From 22e566a1c06ebeb5569c07041d337d32e0aa6063 Mon Sep 17 00:00:00 2001 From: Kamil Danecki Date: Thu, 2 Apr 2026 15:09:56 +0200 Subject: [PATCH] Allow warning in tests Signed-off-by: Kamil Danecki --- test_regress/t/t_event_array_fire.py | 2 +- test_regress/t/t_fork_dynscope_interface.py | 2 +- test_regress/t/t_fork_dynscope_out.py | 2 +- test_regress/t/t_fork_repeat_reset.py | 2 +- test_regress/t/t_fork_write_after_timing.py | 2 +- .../t/t_fork_write_after_timing_bad.out | 5 +++++ test_regress/t/t_timing_func_fork_bad.out | 22 +++++++++---------- test_regress/t/t_uvm_dpi_v2017_1_0.py | 2 +- test_regress/t/t_uvm_dpi_v2020_3_1.py | 2 +- .../t/t_uvm_hello_all_v2017_1_0_dpi.py | 1 + .../t/t_uvm_hello_all_v2020_3_1_dpi.py | 1 + test_regress/t/t_virtual_interface_method.py | 2 +- 12 files changed, 26 insertions(+), 19 deletions(-) diff --git a/test_regress/t/t_event_array_fire.py b/test_regress/t/t_event_array_fire.py index 4ee7f9e14..a6946df89 100755 --- a/test_regress/t/t_event_array_fire.py +++ b/test_regress/t/t_event_array_fire.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('simulator') -test.compile(verilator_flags2=['--binary']) +test.compile(verilator_flags2=['--binary', '-Wno-IMPLICITSTATIC']) test.execute() diff --git a/test_regress/t/t_fork_dynscope_interface.py b/test_regress/t/t_fork_dynscope_interface.py index 6ed2bc5c7..afb502505 100755 --- a/test_regress/t/t_fork_dynscope_interface.py +++ b/test_regress/t/t_fork_dynscope_interface.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('simulator') -test.compile(verilator_flags2=['--binary --trace-fst']) +test.compile(verilator_flags2=['--binary --trace-fst', '-Wno-IMPLICITSTATIC']) test.execute() diff --git a/test_regress/t/t_fork_dynscope_out.py b/test_regress/t/t_fork_dynscope_out.py index f478cb764..e423547dd 100755 --- a/test_regress/t/t_fork_dynscope_out.py +++ b/test_regress/t/t_fork_dynscope_out.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('simulator') -test.compile(verilator_flags2=["--binary -Wno-INITIALDLY"]) +test.compile(verilator_flags2=["--binary -Wno-INITIALDLY", '-Wno-IMPLICITSTATIC']) test.execute() diff --git a/test_regress/t/t_fork_repeat_reset.py b/test_regress/t/t_fork_repeat_reset.py index 31b1f0e53..b3b2ce95d 100755 --- a/test_regress/t/t_fork_repeat_reset.py +++ b/test_regress/t/t_fork_repeat_reset.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('simulator') -test.compile(verilator_flags2=["--binary"]) +test.compile(verilator_flags2=["--binary", '-Wno-IMPLICITSTATIC']) test.execute() diff --git a/test_regress/t/t_fork_write_after_timing.py b/test_regress/t/t_fork_write_after_timing.py index f3eeea08e..044191f3f 100755 --- a/test_regress/t/t_fork_write_after_timing.py +++ b/test_regress/t/t_fork_write_after_timing.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('simulator') -test.compile(verilator_flags2=["--binary", "-Wno-implicitstatic"]) +test.compile(verilator_flags2=["--binary", "-Wno-IMPLICITSTATIC"]) test.execute() diff --git a/test_regress/t/t_fork_write_after_timing_bad.out b/test_regress/t/t_fork_write_after_timing_bad.out index 99c600148..b22afa40a 100644 --- a/test_regress/t/t_fork_write_after_timing_bad.out +++ b/test_regress/t/t_fork_write_after_timing_bad.out @@ -1,3 +1,8 @@ +%Warning-IMPLICITSTATIC: t/t_fork_write_after_timing_bad.v:12:19: Variable's lifetime implicitly set to static because in static task with timing control. + 12 | task t1(ref int x); + | ^ + ... For warning description see https://verilator.org/warn/IMPLICITSTATIC?v=latest + ... Use "/* verilator lint_off IMPLICITSTATIC */" and lint_on around source to disable this message. %Error: t/t_fork_write_after_timing_bad.v:12:19: It is illegal to use argument passing by reference for subroutines with a lifetime of static (IEEE 1800-2023 13.5.2) : ... note: In instance 't' 12 | task t1(ref int x); diff --git a/test_regress/t/t_timing_func_fork_bad.out b/test_regress/t/t_timing_func_fork_bad.out index a605bf784..8e594bf3d 100644 --- a/test_regress/t/t_timing_func_fork_bad.out +++ b/test_regress/t/t_timing_func_fork_bad.out @@ -1,45 +1,45 @@ -%Error: t/t_timing_func_fork_bad.v:13:9: Writing to an output automatic variable of a function after a timing control is not allowed +%Error: t/t_timing_func_fork_bad.v:13:9: Writing to an output automatic variable of a function after a timing control is not allowed (IEEE 1800-2023 13.2.2) : ... note: In instance 't' 13 | f1 = 0; | ^~ ... See the manual at https://verilator.org/verilator_doc.html?v=latest for more assistance. -%Error: t/t_timing_func_fork_bad.v:14:9: Writing to an output automatic variable of a function after a timing control is not allowed +%Error: t/t_timing_func_fork_bad.v:14:9: Writing to an output automatic variable of a function after a timing control is not allowed (IEEE 1800-2023 13.2.2) : ... note: In instance 't' 14 | o1 = 0; | ^~ -%Error: t/t_timing_func_fork_bad.v:22:7: Writing to an output automatic variable of a task after a timing control is not allowed +%Error: t/t_timing_func_fork_bad.v:22:7: Writing to an output automatic variable of a task after a timing control is not allowed (IEEE 1800-2023 13.2.2) : ... note: In instance 't' 22 | o1 = 0; | ^~ -%Error: t/t_timing_func_fork_bad.v:29:9: Writing to an output automatic variable of a function after a timing control is not allowed +%Error: t/t_timing_func_fork_bad.v:29:9: Writing to an output automatic variable of a function after a timing control is not allowed (IEEE 1800-2023 13.2.2) : ... note: In instance 't' 29 | f2 = #5 0; | ^~ -%Error: t/t_timing_func_fork_bad.v:31:9: Writing to an inout automatic variable of a function after a timing control is not allowed +%Error: t/t_timing_func_fork_bad.v:31:9: Writing to an inout automatic variable of a function after a timing control is not allowed (IEEE 1800-2023 13.2.2) : ... note: In instance 't' 31 | io2 = 0; | ^~~ -%Error: t/t_timing_func_fork_bad.v:41:9: Writing to an output automatic variable of a function after a timing control is not allowed +%Error: t/t_timing_func_fork_bad.v:41:9: Writing to an output automatic variable of a function after a timing control is not allowed (IEEE 1800-2023 13.2.2) : ... note: In instance 't' 41 | f3 = 0; | ^~ -%Error: t/t_timing_func_fork_bad.v:42:9: Writing to an output automatic variable of a function after a timing control is not allowed +%Error: t/t_timing_func_fork_bad.v:42:9: Writing to an output automatic variable of a function after a timing control is not allowed (IEEE 1800-2023 13.2.2) : ... note: In instance 't' 42 | o3 = 0; | ^~ -%Error: t/t_timing_func_fork_bad.v:50:9: Writing to an output automatic variable of a function after a timing control is not allowed +%Error: t/t_timing_func_fork_bad.v:50:9: Writing to an output automatic variable of a function after a timing control is not allowed (IEEE 1800-2023 13.2.2) : ... note: In instance 't' 50 | f4 = @e 0; | ^~ -%Error: t/t_timing_func_fork_bad.v:52:9: Writing to an inout automatic variable of a function after a timing control is not allowed +%Error: t/t_timing_func_fork_bad.v:52:9: Writing to an inout automatic variable of a function after a timing control is not allowed (IEEE 1800-2023 13.2.2) : ... note: In instance 't' 52 | io4 = 0; | ^~~ -%Error: t/t_timing_func_fork_bad.v:63:9: Writing to an output automatic variable of a function after a timing control is not allowed +%Error: t/t_timing_func_fork_bad.v:63:9: Writing to an output automatic variable of a function after a timing control is not allowed (IEEE 1800-2023 13.2.2) : ... note: In instance 't' 63 | f5 = 0; | ^~ -%Error: t/t_timing_func_fork_bad.v:64:9: Writing to an output automatic variable of a function after a timing control is not allowed +%Error: t/t_timing_func_fork_bad.v:64:9: Writing to an output automatic variable of a function after a timing control is not allowed (IEEE 1800-2023 13.2.2) : ... note: In instance 't' 64 | o5 = 0; | ^~ diff --git a/test_regress/t/t_uvm_dpi_v2017_1_0.py b/test_regress/t/t_uvm_dpi_v2017_1_0.py index 42e81f4b6..0f0153143 100755 --- a/test_regress/t/t_uvm_dpi_v2017_1_0.py +++ b/test_regress/t/t_uvm_dpi_v2017_1_0.py @@ -17,7 +17,7 @@ if re.search(r'clang', test.cxx_version): test.skip("uvm_regex.cc from upstream has clang warnings") test.compile(verilator_flags2=[ - "--binary", test.build_jobs, "--vpi", "+define+T_V2017_1_0", "+incdir+t/uvm/v2017_1_0", + "--binary", "-Wno-IMPLICITSTATIC", test.build_jobs, "--vpi", "+define+T_V2017_1_0", "+incdir+t/uvm/v2017_1_0", test.pli_filename ]) diff --git a/test_regress/t/t_uvm_dpi_v2020_3_1.py b/test_regress/t/t_uvm_dpi_v2020_3_1.py index 288cde74e..cbe3158a2 100755 --- a/test_regress/t/t_uvm_dpi_v2020_3_1.py +++ b/test_regress/t/t_uvm_dpi_v2020_3_1.py @@ -17,7 +17,7 @@ if re.search(r'clang', test.cxx_version): test.skip("uvm_regex.cc from upstream has clang warnings") test.compile(verilator_flags2=[ - "--binary", test.build_jobs, "--vpi", "+define+T_V2020_3_1", "+incdir+t/uvm/v2020_3_1", + "--binary", test.build_jobs, "--vpi", "+define+T_V2020_3_1", "+incdir+t/uvm/v2020_3_1", "-Wno-IMPLICITSTATIC", test.pli_filename ]) diff --git a/test_regress/t/t_uvm_hello_all_v2017_1_0_dpi.py b/test_regress/t/t_uvm_hello_all_v2017_1_0_dpi.py index c410f628b..e27e4eb85 100755 --- a/test_regress/t/t_uvm_hello_all_v2017_1_0_dpi.py +++ b/test_regress/t/t_uvm_hello_all_v2017_1_0_dpi.py @@ -23,6 +23,7 @@ test.compile(v_flags2=[ "--vpi", "--CFLAGS -O0", "-Wall", + "-Wno-IMPLICITSTATIC", "+incdir+t/uvm", # "t/uvm/uvm_pkg_all_v2017_1_0_dpi.svh", test.pli_filename diff --git a/test_regress/t/t_uvm_hello_all_v2020_3_1_dpi.py b/test_regress/t/t_uvm_hello_all_v2020_3_1_dpi.py index b6ffb1f15..97d73bd14 100755 --- a/test_regress/t/t_uvm_hello_all_v2020_3_1_dpi.py +++ b/test_regress/t/t_uvm_hello_all_v2020_3_1_dpi.py @@ -23,6 +23,7 @@ test.compile(v_flags2=[ "--vpi", "--CFLAGS -O0", "-Wall", + '-Wno-IMPLICITSTATIC', "+incdir+t/uvm", # "t/uvm/uvm_pkg_all_v2020_3_1_dpi.svh", test.pli_filename diff --git a/test_regress/t/t_virtual_interface_method.py b/test_regress/t/t_virtual_interface_method.py index 61fae108a..1ca098664 100755 --- a/test_regress/t/t_virtual_interface_method.py +++ b/test_regress/t/t_virtual_interface_method.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('simulator') -test.compile(v_flags2=["--binary"], verilator_make_gmake=False) +test.compile(v_flags2=["--binary", '-Wno-IMPLICITSTATIC'], verilator_make_gmake=False) test.execute()