luke
/
iverilog
mirror of https://github.com/steveicarus/iverilog.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Eliminate unnecessary rerun of test_width in case statement elaboration. 

Also fix a typo in a comment.
This commit is contained in:
Martin Whitaker 2014-11-02 09:11:54 +00:00
parent 9a471bd81c
commit e38b5d9fb7
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
elaborate.cc
View File

@ -3003,10 +3003,10 @@ NetProc* PCase::elaborate(Design*des, NetScope*scope) const
context_width = 1; context_width = 1;
context_unsigned = false; context_unsigned = false;
} else if (context_mode > PExpr::SIZED) { } else if (context_mode >= PExpr::LOSSLESS) {
/* Expressions may choose a different size if they are /* Expressions may choose a different size if they are
in an unsized context, so we need to run through the in a lossless context, so we need to run through the
process again to get the final expression width. */ process again to get the final expression width. */
context_width = test_case_width(des, scope, expr_, context_mode); context_width = test_case_width(des, scope, expr_, context_mode);
@ -3069,7 +3069,7 @@ NetProc* PCase::elaborate(Design*des, NetScope*scope) const
/* Iterate over all the case items (guard/statement pairs) /* Iterate over all the case items (guard/statement pairs)
elaborating them. If the guard has no expression, then this elaborating them. If the guard has no expression, then this
is a "default" cause. Otherwise, the guard has one or more is a "default" case. Otherwise, the guard has one or more
expressions, and each guard is a case. */ expressions, and each guard is a case. */
unsigned inum = 0; unsigned inum = 0;
for (unsigned idx = 0 ; idx < items_->count() ; idx += 1) { for (unsigned idx = 0 ; idx < items_->count() ; idx += 1) {