diff --git a/bin/verilator b/bin/verilator index 130fb6222..4ef6e274c 100755 --- a/bin/verilator +++ b/bin/verilator @@ -2573,7 +2573,7 @@ cases would result in simulator mismatches. Warns that the specified signal is never sourced. -Ignoring this warning will only suppress the lint check, it will simulate +Disabled by default as this is a code style warning, it will simulate correctly. =item UNOPT @@ -2659,7 +2659,7 @@ correctly. Warns that the specified signal is never sinked. This is a future message, currently Verilator will not produce this warning. -Ignoring this warning will only suppress the lint check, it will simulate +Disabled by default as this is a code style warning, it will simulate correctly. =item VARHIDDEN diff --git a/src/V3Error.h b/src/V3Error.h index ce51c975e..4a7920457 100644 --- a/src/V3Error.h +++ b/src/V3Error.h @@ -132,13 +132,14 @@ public: || m_e==CMPCONST || m_e==IMPLICIT || m_e==LITENDIAN - || m_e==UNDRIVEN || m_e==UNSIGNED - || m_e==UNUSED + || m_e==UNSIGNED || m_e==WIDTH); } // Warnings that are style only bool styleError() const { return ( m_e==DEFPARAM || m_e==DECLFILENAME || m_e==INCABSPATH + || m_e==UNDRIVEN + || m_e==UNUSED || m_e==VARHIDDEN ); } }; inline bool operator== (V3ErrorCode lhs, V3ErrorCode rhs) { return (lhs.m_e == rhs.m_e); } diff --git a/test_regress/t/t_lint_implicit_bad.pl b/test_regress/t/t_lint_implicit_bad.pl index 79949a35b..594ae1d01 100755 --- a/test_regress/t/t_lint_implicit_bad.pl +++ b/test_regress/t/t_lint_implicit_bad.pl @@ -10,7 +10,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di top_filename("t/t_lint_implicit.v"); compile ( - v_flags2 => ["--lint-only"], + v_flags2 => ["--lint-only -Wall -Wno-DECLFILENAME"], fails=>1, expect=> '%Warning-IMPLICIT: t/t_lint_implicit.v:\d+: Signal definition not found, creating implicitly: b diff --git a/test_regress/t/t_lint_implicit_def_bad.pl b/test_regress/t/t_lint_implicit_def_bad.pl index fdb19bd62..de69addfe 100755 --- a/test_regress/t/t_lint_implicit_def_bad.pl +++ b/test_regress/t/t_lint_implicit_def_bad.pl @@ -8,7 +8,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di # Version 2.0. compile ( - v_flags2 => ["--lint-only"], + v_flags2 => ["--lint-only -Wall -Wno-DECLFILENAME"], fails=>1, expect=> '%Warning-IMPLICIT: t/t_lint_implicit_def_bad.v:\d+: Signal definition not found, creating implicitly: imp_warn