From 29db25b70e8b636b61d5c1b840cfd60784b5b3fa Mon Sep 17 00:00:00 2001 From: Geza Lore Date: Sun, 9 Jun 2024 14:39:17 +0100 Subject: [PATCH] Reset radix in width warnings (#5166) --- src/V3Width.cpp | 2 +- test_regress/t/t_select_bad_range4.out | 14 +++++++------- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/src/V3Width.cpp b/src/V3Width.cpp index 0531cae38..d3188dd03 100644 --- a/src/V3Width.cpp +++ b/src/V3Width.cpp @@ -880,7 +880,7 @@ class WidthVisitor final : public VNVisitor { const int width = nodep->elementsConst(); if (width > (1 << 28)) { nodep->v3error("Width of bit range is huge; vector of over 1 billion bits: 0x" - << std::hex << width); + << std::hex << width << std::dec); } // Note width() not set on range; use elementsConst() if (nodep->ascending() && !VN_IS(nodep->backp(), UnpackArrayDType) diff --git a/test_regress/t/t_select_bad_range4.out b/test_regress/t/t_select_bad_range4.out index e2d32618d..917094127 100644 --- a/test_regress/t/t_select_bad_range4.out +++ b/test_regress/t/t_select_bad_range4.out @@ -20,15 +20,15 @@ : ... note: In instance 't' 21 | sel2 = mi[44 +: 1<<29]; | ^ -%Warning-SELRANGE: t/t_select_bad_range4.v:21:16: Extracting 20000000 bits from only 6 bit number +%Warning-SELRANGE: t/t_select_bad_range4.v:21:16: Extracting 536870912 bits from only 6 bit number : ... note: In instance 't' 21 | sel2 = mi[44 +: 1<<29]; | ^ -%Warning-SELRANGE: t/t_select_bad_range4.v:21:16: Selection index out of range: 20000003:4 outside 2d:28 +%Warning-SELRANGE: t/t_select_bad_range4.v:21:16: Selection index out of range: 536870915:4 outside 45:40 : ... note: In instance 't' 21 | sel2 = mi[44 +: 1<<29]; | ^ -%Warning-WIDTHTRUNC: t/t_select_bad_range4.v:21:12: Operator ASSIGN expects 4 bits on the Assign RHS, but Assign RHS's SEL generates 20000000 bits. +%Warning-WIDTHTRUNC: t/t_select_bad_range4.v:21:12: Operator ASSIGN expects 4 bits on the Assign RHS, but Assign RHS's SEL generates 536870912 bits. : ... note: In instance 't' 21 | sel2 = mi[44 +: 1<<29]; | ^ @@ -44,7 +44,7 @@ : ... note: In instance 't' 22 | sel2 = mi[44 +: nonconst]; | ^ -%Warning-WIDTHEXPAND: t/t_select_bad_range4.v:23:17: Operator SUB expects 20 or 6 bits on the LHS, but LHS's VARREF 'nonconst' generates 1 bits. +%Warning-WIDTHEXPAND: t/t_select_bad_range4.v:23:17: Operator SUB expects 32 or 6 bits on the LHS, but LHS's VARREF 'nonconst' generates 1 bits. : ... note: In instance 't' 23 | sel2 = mi[nonconst]; | ^~~~~~~~ @@ -64,15 +64,15 @@ : ... note: In instance 't' 24 | sel2 = mi[nonconst : nonconst]; | ^ -%Warning-SELRANGE: t/t_select_bad_range4.v:25:16: Extracting 20000001 bits from only 6 bit number +%Warning-SELRANGE: t/t_select_bad_range4.v:25:16: Extracting 536870913 bits from only 6 bit number : ... note: In instance 't' 25 | sel2 = mi[1<<29 : 0]; | ^ -%Warning-SELRANGE: t/t_select_bad_range4.v:25:16: Selection index out of range: 1fffffd8:ffffffd8 outside 2d:28 +%Warning-SELRANGE: t/t_select_bad_range4.v:25:16: Selection index out of range: 536870872:-40 outside 45:40 : ... note: In instance 't' 25 | sel2 = mi[1<<29 : 0]; | ^ -%Warning-WIDTHTRUNC: t/t_select_bad_range4.v:25:12: Operator ASSIGN expects 4 bits on the Assign RHS, but Assign RHS's SEL generates 20000001 bits. +%Warning-WIDTHTRUNC: t/t_select_bad_range4.v:25:12: Operator ASSIGN expects 4 bits on the Assign RHS, but Assign RHS's SEL generates 536870913 bits. : ... note: In instance 't' 25 | sel2 = mi[1<<29 : 0]; | ^