From 3c5bf034ca1c2475c74024462b6d0b73e3c21ed5 Mon Sep 17 00:00:00 2001 From: Larry Doolittle Date: Mon, 12 May 2008 09:12:07 -0700 Subject: [PATCH] One more signed vs unsigned comparison fix Fix elab_expr.cc:1561: warning: comparison between signed and unsigned integer expressions caused by revised prototype of NetNet::sb_to_idx() in commit dfb7bf52115e39dc53d48e71847acc9767f1cc47 --- elab_expr.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/elab_expr.cc b/elab_expr.cc index 508617b35..28f0595d0 100644 --- a/elab_expr.cc +++ b/elab_expr.cc @@ -1558,7 +1558,8 @@ NetExpr* PEIdent::elaborate_expr_net_idx_do_(Design*des, NetScope*scope, // If the part select covers exactly the entire // vector, then do not bother with it. Return the // signal itself. - if (net->sig()->sb_to_idx(lsv) == (wid-1) && wid == net->vector_width()) + long l = net->sig()->sb_to_idx(lsv); + if (l >= 0 && (unsigned) l == (wid-1) && wid == net->vector_width()) return net; // Otherwise, make a part select that covers the right range.