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

Fix warning and little E full vector up/down selects

This commit is contained in:
Cary R 2021-01-30 00:34:10 -08:00
parent 31a68fcd8a
commit 0c12344b27
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
elab_expr.cc
View File

@ -5809,6 +5809,7 @@ NetExpr* PEIdent::elaborate_expr_net_idx_up_(Design*des, NetScope*scope,
} }
long rel_base = net->sig()->sb_to_idx(prefix_indices, lsv); long rel_base = net->sig()->sb_to_idx(prefix_indices, lsv);
rel_base += offset;
// If the part select covers exactly the entire // If the part select covers exactly the entire
// vector, then do not bother with it. Return the // vector, then do not bother with it. Return the
@ -5821,7 +5822,7 @@ NetExpr* PEIdent::elaborate_expr_net_idx_up_(Design*des, NetScope*scope,
// Otherwise, make a part select that covers the right // Otherwise, make a part select that covers the right
// range. // range.
ex = new NetEConst(verinum(rel_base + offset)); ex = new NetEConst(verinum(rel_base));
if (warn_ob_select) { if (warn_ob_select) {
if (rel_base < 0) { if (rel_base < 0) {
cerr << get_fileline() << ": warning: " cerr << get_fileline() << ": warning: "
@ -5912,11 +5913,12 @@ NetExpr* PEIdent::elaborate_expr_net_idx_do_(Design*des, NetScope*scope,
} }
long rel_base = net->sig()->sb_to_idx(prefix_indices, lsv); long rel_base = net->sig()->sb_to_idx(prefix_indices, lsv);
rel_base += offset;
// If the part select covers exactly the entire // If the part select covers exactly the entire
// vector, then do not bother with it. Return the // vector, then do not bother with it. Return the
// signal itself. // signal itself.
if (rel_base == (long) (wid-1) && wid == net->vector_width()) { if (rel_base == (long)(wid-1) && wid == net->vector_width()) {
delete base; delete base;
net->cast_signed(false); net->cast_signed(false);
return net; return net;
@ -5924,7 +5926,6 @@ NetExpr* PEIdent::elaborate_expr_net_idx_do_(Design*des, NetScope*scope,
// Otherwise, make a part select that covers the right // Otherwise, make a part select that covers the right
// range. // range.
rel_base += offset;
ex = new NetEConst(verinum(rel_base)); ex = new NetEConst(verinum(rel_base));
if (warn_ob_select) { if (warn_ob_select) {
if (rel_base < 0) { if (rel_base < 0) {