From f9f16b438f0edc2e911fc3bf5def61fd0dab4ed7 Mon Sep 17 00:00:00 2001 From: Wilson Snyder Date: Wed, 10 Dec 2025 19:52:51 -0500 Subject: [PATCH] Tests: Imply --lint-only with test.lint() --- test_regress/driver.py | 11 ++++++++--- test_regress/t/t_a3_selftest_thread.py | 2 +- test_regress/t/t_case_enum_complete.py | 2 +- test_regress/t/t_case_enum_complete_wildcard.py | 2 +- test_regress/t/t_clk_first_bad.py | 2 +- test_regress/t/t_clk_first_deprecated.py | 2 +- test_regress/t/t_concat_or.py | 2 +- test_regress/t/t_config_hier.py | 2 +- test_regress/t/t_config_include_bad.py | 2 +- test_regress/t/t_config_inst_missing.py | 2 +- test_regress/t/t_config_libmap.py | 2 +- test_regress/t/t_config_param.py | 4 +--- test_regress/t/t_covergroup_in_class_duplicate_bad.py | 2 +- test_regress/t/t_covergroup_with_function_foo_bad.py | 2 +- .../t/t_covergroup_with_sample_args_too_few_bad.py | 2 +- .../t/t_covergroup_with_sample_args_too_many_bad.py | 2 +- test_regress/t/t_debug_fatalsrc_bt_bad.py | 2 +- test_regress/t/t_debug_sigsegv_bt_bad.py | 2 +- test_regress/t/t_debug_width.py | 4 +--- test_regress/t/t_disable_func_bad.py | 4 +--- test_regress/t/t_dump_tree_dot.py | 2 +- test_regress/t/t_enum_bad_cell.py | 4 +--- test_regress/t/t_enum_bad_circdecl.py | 4 +--- test_regress/t/t_enum_bad_dup.py | 4 +--- test_regress/t/t_enum_bad_hide.py | 4 +--- test_regress/t/t_enum_bad_wrap.py | 4 +--- test_regress/t/t_flag_build_jobs_bad.out | 1 - test_regress/t/t_flag_build_jobs_bad.py | 2 +- test_regress/t/t_flag_f_bad.py | 2 +- test_regress/t/t_flag_main_sc_bad.py | 6 +++--- test_regress/t/t_flag_only_bad.py | 9 ++++----- test_regress/t/t_flag_only_bad2.py | 2 +- test_regress/t/t_flag_only_bad3.py | 9 ++++----- test_regress/t/t_flag_verilate_threads_bad.py | 4 +++- test_regress/t/t_flag_wfatal.py | 2 +- test_regress/t/t_flag_woff_bad.py | 2 +- test_regress/t/t_fork_bbox.py | 2 +- test_regress/t/t_fork_func2_bad.py | 2 +- test_regress/t/t_fork_func_bad.py | 2 +- test_regress/t/t_func_begin2.py | 2 +- test_regress/t/t_hier_block1_bad.py | 2 +- test_regress/t/t_hier_block_threads_bad.py | 8 ++++---- test_regress/t/t_inst_missing.py | 2 +- test_regress/t/t_inst_missing_bad.py | 2 +- test_regress/t/t_lint_always_comb_multidriven_bad.py | 2 +- ...always_comb_multidriven_compile_public_flat_bad.py | 4 +--- test_regress/t/t_lint_blkseq_bad.py | 2 +- test_regress/t/t_lint_blkseq_loop.py | 2 +- test_regress/t/t_lint_comb_use.py | 2 +- test_regress/t/t_lint_declfilename_bad.py | 2 +- test_regress/t/t_lint_declfilename_bbox.py | 2 +- test_regress/t/t_lint_defparam_bad.py | 2 +- test_regress/t/t_lint_eofnewline.py | 2 +- test_regress/t/t_lint_eofnewline_bad.py | 2 +- test_regress/t/t_lint_eofnewline_vlt.py | 2 +- test_regress/t/t_lint_genunnamed_bad.py | 2 +- test_regress/t/t_lint_historical.py | 2 +- test_regress/t/t_lint_ifdepth_bad.py | 2 +- test_regress/t/t_lint_implicit_bad.py | 4 +--- test_regress/t/t_lint_implicit_def_bad.py | 2 +- test_regress/t/t_lint_implicit_func_bad.py | 2 +- test_regress/t/t_lint_implicit_type_bad.py | 2 +- test_regress/t/t_lint_import_name2_bad.py | 2 +- test_regress/t/t_lint_import_name_bad.py | 2 +- test_regress/t/t_lint_importstar_bad.py | 2 +- test_regress/t/t_lint_incabspath_bad.py | 2 +- test_regress/t/t_lint_latch_bad.py | 2 +- test_regress/t/t_lint_lint_bad.py | 4 +--- test_regress/t/t_lint_lint_no.py | 2 +- test_regress/t/t_lint_misindent_bad.py | 2 +- test_regress/t/t_lint_mod_paren_bad.py | 4 +--- test_regress/t/t_lint_modport_dir_bad.py | 4 +--- test_regress/t/t_lint_once_bad.py | 4 +--- test_regress/t/t_lint_pragma_protected.py | 2 +- test_regress/t/t_lint_pragma_protected_bad.py | 4 +--- test_regress/t/t_lint_realcvt_bad.py | 4 +--- test_regress/t/t_lint_setout_bad_noinl.py | 4 +--- test_regress/t/t_lint_style_bad.py | 4 +--- test_regress/t/t_lint_style_no.py | 2 +- test_regress/t/t_lint_syncasyncnet_bad.py | 2 +- test_regress/t/t_lint_unsup_deassign.py | 2 +- test_regress/t/t_lint_unsup_mixed.py | 2 +- test_regress/t/t_lint_unused.py | 2 +- test_regress/t/t_lint_unused_iface.py | 2 +- test_regress/t/t_lint_unused_iface_bad.py | 2 +- test_regress/t/t_lint_unused_tri.py | 2 +- test_regress/t/t_lint_widthexpand_docs_bad.py | 4 +--- test_regress/t/t_no_std_bad.py | 4 +--- test_regress/t/t_no_std_pkg_bad.py | 2 +- test_regress/t/t_order_blkloopinit_bad.py | 2 +- test_regress/t/t_param_default_presv_bad.py | 2 +- test_regress/t/t_param_sel_range.py | 2 +- test_regress/t/t_pipe_exit_bad.py | 3 ++- test_regress/t/t_pp_pragma_bad.py | 2 +- test_regress/t/t_process_bad.py | 6 +++--- test_regress/t/t_randomize_complex_member_bad.py | 2 +- test_regress/t/t_sarif.sarif.out | 2 +- test_regress/t/t_std_process_self.py | 2 +- test_regress/t/t_std_process_self_std.py | 2 +- test_regress/t/t_std_randomize_mod.py | 2 +- test_regress/t/t_std_randomize_no_args.py | 2 +- test_regress/t/t_timescale_lint.py | 2 +- test_regress/t/t_timescale_lint2.py | 2 +- test_regress/t/t_timescale_lint_bad.py | 2 +- test_regress/t/t_trace_multi_bad.py | 2 +- test_regress/t/t_typedef_no_bad.py | 2 +- test_regress/t/t_udp_bad.py | 2 +- test_regress/t/t_udp_tableeof_bad.py | 2 +- test_regress/t/t_var_bad_hide.py | 4 +--- test_regress/t/t_var_bad_hide2.py | 2 +- test_regress/t/t_var_bad_hide_docs.py | 2 +- test_regress/t/t_var_in_assign_bad.py | 2 +- test_regress/t/t_varref_scope_in_interface.py | 2 +- test_regress/t/t_virtual_interface_method_bad.py | 2 +- test_regress/t/t_vlt_match_contents.py | 2 +- test_regress/t/t_vlt_match_error_1.py | 9 ++++----- test_regress/t/t_vlt_match_error_2.py | 9 ++++----- test_regress/t/t_vlt_match_error_3.py | 9 ++++----- test_regress/t/t_vlt_warn.py | 2 +- test_regress/t/t_vlt_warn_bad.py | 2 +- test_regress/t/t_vlt_warn_ecode_bad.py | 2 +- test_regress/t/t_wait.py | 4 +--- test_regress/t/t_wire_beh1364_bad.py | 2 +- test_regress/t/t_wire_beh1800_bad.py | 2 +- test_regress/t/t_wire_behp1364_bad.py | 2 +- test_regress/t/t_wire_behp1800_bad.py | 2 +- test_regress/t/t_wire_self_bad.py | 2 +- test_regress/t/t_with_suggest_bad.py | 2 +- test_regress/t/t_xml_deprecated_bad.py | 6 +++--- 129 files changed, 162 insertions(+), 204 deletions(-) diff --git a/test_regress/driver.py b/test_regress/driver.py index 674727a92..235d37973 100755 --- a/test_regress/driver.py +++ b/test_regress/driver.py @@ -859,7 +859,7 @@ class VlTest: ] # Verilator self.verilator_define = 'VERILATOR' - self.verilator_flags = [ + self.verilator_flags = [ # See also override (search for -lint-only) "-cc", "-Mdir", self.obj_dir, @@ -1183,10 +1183,15 @@ class VlTest: """Run a linter. Arguments similar to run(); default arguments are from self""" param = {} param.update(vars(self)) - param.update({ # Lint-specific default overrides + param.update({ # Lint-specific default overrides (search also for -cc) 'make_main': False, 'make_top_shell': False, - 'verilator_flags2': ["--lint-only"], + 'verilator_flags': [ + "--lint-only", + "-Mdir", + self.obj_dir, + "--debug-check", + ], 'verilator_make_gmake': False }) param.update(kwargs) diff --git a/test_regress/t/t_a3_selftest_thread.py b/test_regress/t/t_a3_selftest_thread.py index 9ad2150da..a20a9e981 100755 --- a/test_regress/t/t_a3_selftest_thread.py +++ b/test_regress/t/t_a3_selftest_thread.py @@ -13,6 +13,6 @@ test.priority(100) test.scenarios('vltmt') test.top_filename = "t/t_EXAMPLE.v" -test.lint(v_flags=["--lint-only --verilate-jobs 2 --debug-self-test"]) +test.lint(v_flags=["--verilate-jobs 2 --debug-self-test"]) test.passes() diff --git a/test_regress/t/t_case_enum_complete.py b/test_regress/t/t_case_enum_complete.py index 03bb69a41..07d93ccf0 100755 --- a/test_regress/t/t_case_enum_complete.py +++ b/test_regress/t/t_case_enum_complete.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wwarn-CASEINCOMPLETE"]) +test.lint(verilator_flags2=["-Wwarn-CASEINCOMPLETE"]) test.passes() diff --git a/test_regress/t/t_case_enum_complete_wildcard.py b/test_regress/t/t_case_enum_complete_wildcard.py index 03bb69a41..07d93ccf0 100755 --- a/test_regress/t/t_case_enum_complete_wildcard.py +++ b/test_regress/t/t_case_enum_complete_wildcard.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wwarn-CASEINCOMPLETE"]) +test.lint(verilator_flags2=["-Wwarn-CASEINCOMPLETE"]) test.passes() diff --git a/test_regress/t/t_clk_first_bad.py b/test_regress/t/t_clk_first_bad.py index d6fa215bd..d087349a1 100755 --- a/test_regress/t/t_clk_first_bad.py +++ b/test_regress/t/t_clk_first_bad.py @@ -12,6 +12,6 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = "t/t_clk_first_deprecated.v" -test.lint(verilator_flags2=["--lint-only"], fails=True, expect_filename=test.golden_filename) +test.lint(fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_clk_first_deprecated.py b/test_regress/t/t_clk_first_deprecated.py index c6aee9ea5..f87d01c5b 100755 --- a/test_regress/t/t_clk_first_deprecated.py +++ b/test_regress/t/t_clk_first_deprecated.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only", "-Wno-DEPRECATED"]) +test.lint(verilator_flags2=["-Wno-DEPRECATED"]) test.passes() diff --git a/test_regress/t/t_concat_or.py b/test_regress/t/t_concat_or.py index f92eef979..c2d985114 100755 --- a/test_regress/t/t_concat_or.py +++ b/test_regress/t/t_concat_or.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only"]) +test.lint() test.passes() diff --git a/test_regress/t/t_config_hier.py b/test_regress/t/t_config_hier.py index 5a6daa51c..d595a0443 100755 --- a/test_regress/t/t_config_hier.py +++ b/test_regress/t/t_config_hier.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only", "--top cfg21"], +test.lint(verilator_flags2=["--top cfg21"], fails=test.vlt_all, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_config_include_bad.py b/test_regress/t/t_config_include_bad.py index cf71fd2b8..31228c9a7 100755 --- a/test_regress/t/t_config_include_bad.py +++ b/test_regress/t/t_config_include_bad.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only"], fails=True, expect_filename=test.golden_filename) +test.lint(fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_config_inst_missing.py b/test_regress/t/t_config_inst_missing.py index 19cf62743..c920a018c 100755 --- a/test_regress/t/t_config_inst_missing.py +++ b/test_regress/t/t_config_inst_missing.py @@ -12,7 +12,7 @@ import vltest_bootstrap test.scenarios('simulator') test.lint(verilator_flags2=[ - '--binary', '--top cfg1', '--work liba', 't/t_config_work__liba.v', '--work libb', + '--top cfg1', '--work liba', 't/t_config_work__liba.v', '--work libb', 't/t_config_work__libb.v' ], fails=test.vlt_all, diff --git a/test_regress/t/t_config_libmap.py b/test_regress/t/t_config_libmap.py index 14bcdb8d6..4438b7436 100755 --- a/test_regress/t/t_config_libmap.py +++ b/test_regress/t/t_config_libmap.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only", "t/" + test.name + ".map"], +test.lint(verilator_flags2=["t/" + test.name + ".map"], fails=test.vlt_all, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_config_param.py b/test_regress/t/t_config_param.py index 60b7d73ba..6a0214ee8 100755 --- a/test_regress/t/t_config_param.py +++ b/test_regress/t/t_config_param.py @@ -11,9 +11,7 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only"], - fails=test.vlt_all, - expect_filename=test.golden_filename) +test.lint(fails=test.vlt_all, expect_filename=test.golden_filename) # Sort so that 'initial' scheduling order is not relevant # test.files_identical_sorted(test.run_log_filename, test.golden_filename, is_logfile=True) diff --git a/test_regress/t/t_covergroup_in_class_duplicate_bad.py b/test_regress/t/t_covergroup_in_class_duplicate_bad.py index dff5516bf..59bdf12b5 100755 --- a/test_regress/t/t_covergroup_in_class_duplicate_bad.py +++ b/test_regress/t/t_covergroup_in_class_duplicate_bad.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('simulator') -test.compile(fails=True, expect_filename=test.golden_filename) +test.lint(fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_covergroup_with_function_foo_bad.py b/test_regress/t/t_covergroup_with_function_foo_bad.py index dff5516bf..59bdf12b5 100755 --- a/test_regress/t/t_covergroup_with_function_foo_bad.py +++ b/test_regress/t/t_covergroup_with_function_foo_bad.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('simulator') -test.compile(fails=True, expect_filename=test.golden_filename) +test.lint(fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_covergroup_with_sample_args_too_few_bad.py b/test_regress/t/t_covergroup_with_sample_args_too_few_bad.py index dff5516bf..59bdf12b5 100755 --- a/test_regress/t/t_covergroup_with_sample_args_too_few_bad.py +++ b/test_regress/t/t_covergroup_with_sample_args_too_few_bad.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('simulator') -test.compile(fails=True, expect_filename=test.golden_filename) +test.lint(fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_covergroup_with_sample_args_too_many_bad.py b/test_regress/t/t_covergroup_with_sample_args_too_many_bad.py index dff5516bf..59bdf12b5 100755 --- a/test_regress/t/t_covergroup_with_sample_args_too_many_bad.py +++ b/test_regress/t/t_covergroup_with_sample_args_too_many_bad.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('simulator') -test.compile(fails=True, expect_filename=test.golden_filename) +test.lint(fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_debug_fatalsrc_bt_bad.py b/test_regress/t/t_debug_fatalsrc_bt_bad.py index 182a73796..aa8c484fb 100755 --- a/test_regress/t/t_debug_fatalsrc_bt_bad.py +++ b/test_regress/t/t_debug_fatalsrc_bt_bad.py @@ -16,7 +16,7 @@ if 'VERILATOR_TEST_NO_GDB' in os.environ: if not test.have_gdb: test.skip("No gdb installed") -test.lint(verilator_flags2=["--lint-only --debug --gdbbt --debug-fatalsrc"], fails='any') +test.lint(verilator_flags2=["--debug --gdbbt --debug-fatalsrc"], fails='any') test.file_grep(test.compile_log_filename, r'%Error: Internal Error: .*: --debug-fatal-src') test.file_grep(test.compile_log_filename, r'See the manual') diff --git a/test_regress/t/t_debug_sigsegv_bt_bad.py b/test_regress/t/t_debug_sigsegv_bt_bad.py index 17d6bfc52..da77759a5 100755 --- a/test_regress/t/t_debug_sigsegv_bt_bad.py +++ b/test_regress/t/t_debug_sigsegv_bt_bad.py @@ -16,7 +16,7 @@ if 'VERILATOR_TEST_NO_GDB' in os.environ: if not test.have_gdb: test.skip("No gdb installed") -test.lint(verilator_flags2=["--lint-only --debug --gdbbt --debug-sigsegv"], fails='any') +test.lint(verilator_flags2=["--debug --gdbbt --debug-sigsegv"], fails='any') test.file_grep(test.compile_log_filename, r'Program received signal SIGSEGV') test.file_grep(test.compile_log_filename, r'in V3Options::') diff --git a/test_regress/t/t_debug_width.py b/test_regress/t/t_debug_width.py index deccc7ea3..6a7313070 100755 --- a/test_regress/t/t_debug_width.py +++ b/test_regress/t/t_debug_width.py @@ -12,8 +12,6 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = "t/t_opt_const.v" -test.lint(verilator_flags2=["--lint-only", "--debug-width"], - fails=True, - expect_filename=test.golden_filename) +test.lint(verilator_flags2=["--debug-width"], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_disable_func_bad.py b/test_regress/t/t_disable_func_bad.py index cf9cba66e..5df24a780 100755 --- a/test_regress/t/t_disable_func_bad.py +++ b/test_regress/t/t_disable_func_bad.py @@ -11,8 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=['--lint-only --timing'], - fails=True, - expect_filename=test.golden_filename) +test.lint(verilator_flags2=['--timing'], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_dump_tree_dot.py b/test_regress/t/t_dump_tree_dot.py index 70a3a8d47..e840dded0 100755 --- a/test_regress/t/t_dump_tree_dot.py +++ b/test_regress/t/t_dump_tree_dot.py @@ -12,6 +12,6 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = "t/t_EXAMPLE.v" -test.lint(v_flags=["--lint-only --dump-tree-dot"]) +test.lint(v_flags=["--dump-tree-dot"]) test.passes() diff --git a/test_regress/t/t_enum_bad_cell.py b/test_regress/t/t_enum_bad_cell.py index 2076da382..84ba24c85 100755 --- a/test_regress/t/t_enum_bad_cell.py +++ b/test_regress/t/t_enum_bad_cell.py @@ -11,8 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wwarn-VARHIDDEN"], - fails=True, - expect_filename=test.golden_filename) +test.lint(verilator_flags2=["-Wwarn-VARHIDDEN"], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_enum_bad_circdecl.py b/test_regress/t/t_enum_bad_circdecl.py index 2076da382..84ba24c85 100755 --- a/test_regress/t/t_enum_bad_circdecl.py +++ b/test_regress/t/t_enum_bad_circdecl.py @@ -11,8 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wwarn-VARHIDDEN"], - fails=True, - expect_filename=test.golden_filename) +test.lint(verilator_flags2=["-Wwarn-VARHIDDEN"], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_enum_bad_dup.py b/test_regress/t/t_enum_bad_dup.py index 2076da382..84ba24c85 100755 --- a/test_regress/t/t_enum_bad_dup.py +++ b/test_regress/t/t_enum_bad_dup.py @@ -11,8 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wwarn-VARHIDDEN"], - fails=True, - expect_filename=test.golden_filename) +test.lint(verilator_flags2=["-Wwarn-VARHIDDEN"], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_enum_bad_hide.py b/test_regress/t/t_enum_bad_hide.py index 2076da382..84ba24c85 100755 --- a/test_regress/t/t_enum_bad_hide.py +++ b/test_regress/t/t_enum_bad_hide.py @@ -11,8 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wwarn-VARHIDDEN"], - fails=True, - expect_filename=test.golden_filename) +test.lint(verilator_flags2=["-Wwarn-VARHIDDEN"], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_enum_bad_wrap.py b/test_regress/t/t_enum_bad_wrap.py index 2076da382..84ba24c85 100755 --- a/test_regress/t/t_enum_bad_wrap.py +++ b/test_regress/t/t_enum_bad_wrap.py @@ -11,8 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wwarn-VARHIDDEN"], - fails=True, - expect_filename=test.golden_filename) +test.lint(verilator_flags2=["-Wwarn-VARHIDDEN"], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_flag_build_jobs_bad.out b/test_regress/t/t_flag_build_jobs_bad.out index bb8cfeeae..0f7b9c536 100644 --- a/test_regress/t/t_flag_build_jobs_bad.out +++ b/test_regress/t/t_flag_build_jobs_bad.out @@ -1,4 +1,3 @@ %Error: --build-jobs requires a non-negative integer, but '-1' was passed ... See the manual at https://verilator.org/verilator_doc.html?v=latest for more assistance. -%Error: The following cannot be used together: --build, --lint-only. Suggest see manual %Error: Exiting due to diff --git a/test_regress/t/t_flag_build_jobs_bad.py b/test_regress/t/t_flag_build_jobs_bad.py index d944bef02..8cb6ddd51 100755 --- a/test_regress/t/t_flag_build_jobs_bad.py +++ b/test_regress/t/t_flag_build_jobs_bad.py @@ -13,7 +13,7 @@ test.scenarios('vlt') test.top_filename = "t/t_flag_werror.v" test.lint(fails=True, - verilator_flags=["--build-jobs -1 --build"], + verilator_flags=["--cc --build-jobs -1 --build"], expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_flag_f_bad.py b/test_regress/t/t_flag_f_bad.py index 9d256232c..3112b79ad 100755 --- a/test_regress/t/t_flag_f_bad.py +++ b/test_regress/t/t_flag_f_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only -f file_will_not_exist.vc"], +test.lint(verilator_flags2=["-f file_will_not_exist.vc"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_flag_main_sc_bad.py b/test_regress/t/t_flag_main_sc_bad.py index 08d370745..6912ff255 100755 --- a/test_regress/t/t_flag_main_sc_bad.py +++ b/test_regress/t/t_flag_main_sc_bad.py @@ -11,8 +11,8 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=['--exe --build --main --sc'], - fails=True, - expect_filename=test.golden_filename) +test.compile(verilator_flags2=['--exe --build --main --sc'], + fails=True, + expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_flag_only_bad.py b/test_regress/t/t_flag_only_bad.py index f440bf973..ebce0ba4b 100755 --- a/test_regress/t/t_flag_only_bad.py +++ b/test_regress/t/t_flag_only_bad.py @@ -12,10 +12,9 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = "t/t_flag_main.v" -test.lint(verilator_flags2=[ - "-Wno-DEPRECATED --binary -E --dpi-hdr-only --lint-only --xml-only --json-only -Wall" -], - fails=True, - expect_filename=test.golden_filename) +test.lint( + verilator_flags2=["-Wno-DEPRECATED --binary -E --dpi-hdr-only --xml-only --json-only -Wall"], + fails=True, + expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_flag_only_bad2.py b/test_regress/t/t_flag_only_bad2.py index 255bd8622..98e0fe1aa 100755 --- a/test_regress/t/t_flag_only_bad2.py +++ b/test_regress/t/t_flag_only_bad2.py @@ -13,7 +13,7 @@ test.scenarios('vlt') test.top_filename = "t/t_flag_main.v" test.lint(verilator_flags2=[ - "-Wno-DEPRECATED --build -E -Wno-fatal --dpi-hdr-only --lint-only --xml-only --json-only" + "-Wno-DEPRECATED --build -E -Wno-fatal --dpi-hdr-only --xml-only --json-only" ], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_flag_only_bad3.py b/test_regress/t/t_flag_only_bad3.py index 397085a9a..6f0ee1fe6 100755 --- a/test_regress/t/t_flag_only_bad3.py +++ b/test_regress/t/t_flag_only_bad3.py @@ -12,10 +12,9 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = "t/t_flag_main.v" -test.lint(verilator_flags2=[ - "-Wall -Wno-DEPRECATED --lint-only -Wno-fatal --dpi-hdr-only --xml-only --json-only" -], - fails=True, - expect_filename=test.golden_filename) +test.lint( + verilator_flags2=["-Wall -Wno-DEPRECATED -Wno-fatal --dpi-hdr-only --xml-only --json-only"], + fails=True, + expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_flag_verilate_threads_bad.py b/test_regress/t/t_flag_verilate_threads_bad.py index 26f50c37b..ec9afe52a 100755 --- a/test_regress/t/t_flag_verilate_threads_bad.py +++ b/test_regress/t/t_flag_verilate_threads_bad.py @@ -12,6 +12,8 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = "t/t_flag_werror.v" -test.lint(fails=True, verilator_flags=["--verilate-jobs -1"], expect_filename=test.golden_filename) +test.lint(fails=True, + verilator_flags=["--lint-only --verilate-jobs -1"], + expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_flag_wfatal.py b/test_regress/t/t_flag_wfatal.py index a23be6d7a..02fa078ff 100755 --- a/test_regress/t/t_flag_wfatal.py +++ b/test_regress/t/t_flag_wfatal.py @@ -12,6 +12,6 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = "t/t_flag_wfatal.v" -test.lint(verilator_flags2=["--lint-only -Wno-fatal"], expect_filename=test.golden_filename) +test.lint(verilator_flags2=["-Wno-fatal"], expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_flag_woff_bad.py b/test_regress/t/t_flag_woff_bad.py index 10a2221a7..23a96f684 100755 --- a/test_regress/t/t_flag_woff_bad.py +++ b/test_regress/t/t_flag_woff_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only -Wno-NOSUCHERRORASTHIS"], +test.lint(verilator_flags2=["-Wno-NOSUCHERRORASTHIS"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_fork_bbox.py b/test_regress/t/t_fork_bbox.py index 7a642a939..0c1f1c71a 100755 --- a/test_regress/t/t_fork_bbox.py +++ b/test_regress/t/t_fork_bbox.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=['--lint-only --no-timing --bbox-unsup']) +test.lint(verilator_flags2=['--no-timing --bbox-unsup']) test.passes() diff --git a/test_regress/t/t_fork_func2_bad.py b/test_regress/t/t_fork_func2_bad.py index d6dde1012..31228c9a7 100755 --- a/test_regress/t/t_fork_func2_bad.py +++ b/test_regress/t/t_fork_func2_bad.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=['--lint-only'], fails=True, expect_filename=test.golden_filename) +test.lint(fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_fork_func_bad.py b/test_regress/t/t_fork_func_bad.py index d6dde1012..31228c9a7 100755 --- a/test_regress/t/t_fork_func_bad.py +++ b/test_regress/t/t_fork_func_bad.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=['--lint-only'], fails=True, expect_filename=test.golden_filename) +test.lint(fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_func_begin2.py b/test_regress/t/t_func_begin2.py index 713453910..93ef8a8f0 100755 --- a/test_regress/t/t_func_begin2.py +++ b/test_regress/t/t_func_begin2.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only --inline-mult 1"]) +test.lint(verilator_flags2=["--inline-mult 1"]) test.passes() diff --git a/test_regress/t/t_hier_block1_bad.py b/test_regress/t/t_hier_block1_bad.py index c80415ffe..ef7f2d674 100755 --- a/test_regress/t/t_hier_block1_bad.py +++ b/test_regress/t/t_hier_block1_bad.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(fails=True, verilator_flags2=['--hierarchical'], expect_filename=test.golden_filename) +test.compile(fails=True, verilator_flags2=['--hierarchical'], expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_hier_block_threads_bad.py b/test_regress/t/t_hier_block_threads_bad.py index 1d1890dcd..d766e4db1 100755 --- a/test_regress/t/t_hier_block_threads_bad.py +++ b/test_regress/t/t_hier_block_threads_bad.py @@ -11,9 +11,9 @@ import vltest_bootstrap test.scenarios('vltmt') -test.lint(fails=True, - verilator_flags2=['t/t_hier_block_threads_bad.vlt', '-DWORKERS=8', '--hierarchical'], - expect_filename=test.golden_filename, - threads=4) +test.compile(fails=True, + verilator_flags2=['t/t_hier_block_threads_bad.vlt', '-DWORKERS=8', '--hierarchical'], + expect_filename=test.golden_filename, + threads=4) test.passes() diff --git a/test_regress/t/t_inst_missing.py b/test_regress/t/t_inst_missing.py index eed023f4a..7fd4e37fd 100755 --- a/test_regress/t/t_inst_missing.py +++ b/test_regress/t/t_inst_missing.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only --Wall -Wno-DECLFILENAME"]) +test.lint(verilator_flags2=["--Wall -Wno-DECLFILENAME"]) test.passes() diff --git a/test_regress/t/t_inst_missing_bad.py b/test_regress/t/t_inst_missing_bad.py index af59f8be9..8641f823f 100755 --- a/test_regress/t/t_inst_missing_bad.py +++ b/test_regress/t/t_inst_missing_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only --Wall -Wno-DECLFILENAME"], +test.lint(verilator_flags2=["--Wall -Wno-DECLFILENAME"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_lint_always_comb_multidriven_bad.py b/test_regress/t/t_lint_always_comb_multidriven_bad.py index d6dde1012..31228c9a7 100755 --- a/test_regress/t/t_lint_always_comb_multidriven_bad.py +++ b/test_regress/t/t_lint_always_comb_multidriven_bad.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=['--lint-only'], fails=True, expect_filename=test.golden_filename) +test.lint(fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_lint_always_comb_multidriven_compile_public_flat_bad.py b/test_regress/t/t_lint_always_comb_multidriven_compile_public_flat_bad.py index 07d3c7396..951c00f44 100755 --- a/test_regress/t/t_lint_always_comb_multidriven_compile_public_flat_bad.py +++ b/test_regress/t/t_lint_always_comb_multidriven_compile_public_flat_bad.py @@ -12,8 +12,6 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = "t/t_lint_always_comb_multidriven_bad.v" -test.lint(verilator_flags2=['--public-flat-rw --lint-only'], - fails=True, - expect_filename=test.golden_filename) +test.lint(verilator_flags2=['--public-flat-rw'], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_lint_blkseq_bad.py b/test_regress/t/t_lint_blkseq_bad.py index f27c3eae1..4adc6df42 100755 --- a/test_regress/t/t_lint_blkseq_bad.py +++ b/test_regress/t/t_lint_blkseq_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wwarn-BLKSEQ -Wwarn-COMBDLY"], +test.lint(verilator_flags2=["-Wwarn-BLKSEQ -Wwarn-COMBDLY"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_lint_blkseq_loop.py b/test_regress/t/t_lint_blkseq_loop.py index e822338c0..ebbf08d1f 100755 --- a/test_regress/t/t_lint_blkseq_loop.py +++ b/test_regress/t/t_lint_blkseq_loop.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only -Wwarn-BLKSEQ -Wwarn-COMBDLY"]) +test.lint(verilator_flags2=["-Wwarn-BLKSEQ -Wwarn-COMBDLY"]) test.passes() diff --git a/test_regress/t/t_lint_comb_use.py b/test_regress/t/t_lint_comb_use.py index b39e4b760..ca24d5b16 100755 --- a/test_regress/t/t_lint_comb_use.py +++ b/test_regress/t/t_lint_comb_use.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only --bbox-sys"]) +test.lint(verilator_flags2=["--bbox-sys"]) test.passes() diff --git a/test_regress/t/t_lint_declfilename_bad.py b/test_regress/t/t_lint_declfilename_bad.py index 8877eb52f..96e0bed25 100755 --- a/test_regress/t/t_lint_declfilename_bad.py +++ b/test_regress/t/t_lint_declfilename_bad.py @@ -12,6 +12,6 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = "t/t_lint_declfilename.v" -test.lint(verilator_flags2=["--lint-only -Wall"], fails=True, expect_filename=test.golden_filename) +test.lint(verilator_flags2=["-Wall"], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_lint_declfilename_bbox.py b/test_regress/t/t_lint_declfilename_bbox.py index f31da0fcd..e037f35c6 100755 --- a/test_regress/t/t_lint_declfilename_bbox.py +++ b/test_regress/t/t_lint_declfilename_bbox.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only -Wall"]) +test.lint(verilator_flags2=["-Wall"]) test.passes() diff --git a/test_regress/t/t_lint_defparam_bad.py b/test_regress/t/t_lint_defparam_bad.py index a707e9afc..c25ef765a 100755 --- a/test_regress/t/t_lint_defparam_bad.py +++ b/test_regress/t/t_lint_defparam_bad.py @@ -12,7 +12,7 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = "t/t_lint_defparam.v" -test.lint(verilator_flags2=["--lint-only -Wwarn-style -Wno-DECLFILENAME"], +test.lint(verilator_flags2=["-Wwarn-style -Wno-DECLFILENAME"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_lint_eofnewline.py b/test_regress/t/t_lint_eofnewline.py index 7f0f9a1c3..64dde988b 100755 --- a/test_regress/t/t_lint_eofnewline.py +++ b/test_regress/t/t_lint_eofnewline.py @@ -21,6 +21,6 @@ def gen(filename): gen(test.top_filename) -test.lint(verilator_flags2=["-E -Wall -Wno-DECLFILENAME"], expect_filename=test.golden_filename) +test.lint(verilator_flags=["-E -Wall -Wno-DECLFILENAME"], expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_lint_eofnewline_bad.py b/test_regress/t/t_lint_eofnewline_bad.py index a3278118b..86acef286 100755 --- a/test_regress/t/t_lint_eofnewline_bad.py +++ b/test_regress/t/t_lint_eofnewline_bad.py @@ -24,7 +24,7 @@ def gen(filename): gen(test.top_filename) -test.lint(verilator_flags2=["--lint-only -Wall -Wno-DECLFILENAME"], +test.lint(verilator_flags2=["-Wall -Wno-DECLFILENAME"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_lint_eofnewline_vlt.py b/test_regress/t/t_lint_eofnewline_vlt.py index b56a9c954..852be2ccd 100755 --- a/test_regress/t/t_lint_eofnewline_vlt.py +++ b/test_regress/t/t_lint_eofnewline_vlt.py @@ -24,6 +24,6 @@ def gen(filename): gen(test.top_filename) -test.lint(verilator_flags2=["--lint-only -Wall -Wno-DECLFILENAME t/t_lint_eofnewline_vlt.vlt"]) +test.lint(verilator_flags2=["-Wall -Wno-DECLFILENAME t/t_lint_eofnewline_vlt.vlt"]) test.passes() diff --git a/test_regress/t/t_lint_genunnamed_bad.py b/test_regress/t/t_lint_genunnamed_bad.py index f05a78df6..946e0710c 100755 --- a/test_regress/t/t_lint_genunnamed_bad.py +++ b/test_regress/t/t_lint_genunnamed_bad.py @@ -12,7 +12,7 @@ import vltest_bootstrap test.scenarios('vlt') test.lint(fails=True, - verilator_flags2=["--lint-only -Wall -Wno-DECLFILENAME"], + verilator_flags2=["-Wall -Wno-DECLFILENAME"], expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_lint_historical.py b/test_regress/t/t_lint_historical.py index 9ab47796d..f81c3d68d 100755 --- a/test_regress/t/t_lint_historical.py +++ b/test_regress/t/t_lint_historical.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only"]) +test.lint() test.passes() diff --git a/test_regress/t/t_lint_ifdepth_bad.py b/test_regress/t/t_lint_ifdepth_bad.py index 2d9f441cf..036372ce1 100755 --- a/test_regress/t/t_lint_ifdepth_bad.py +++ b/test_regress/t/t_lint_ifdepth_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only -Wall -Wno-DECLFILENAME --if-depth 10"], +test.lint(verilator_flags2=["-Wall -Wno-DECLFILENAME --if-depth 10"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_lint_implicit_bad.py b/test_regress/t/t_lint_implicit_bad.py index 3713dce09..fcdd3f4e7 100755 --- a/test_regress/t/t_lint_implicit_bad.py +++ b/test_regress/t/t_lint_implicit_bad.py @@ -12,8 +12,6 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = "t/t_lint_implicit.v" -test.lint(verilator_flags2=["--lint-only -Wwarn-IMPLICIT"], - fails=True, - expect_filename=test.golden_filename) +test.lint(verilator_flags2=["-Wwarn-IMPLICIT"], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_lint_implicit_def_bad.py b/test_regress/t/t_lint_implicit_def_bad.py index c0608225e..0db29c160 100755 --- a/test_regress/t/t_lint_implicit_def_bad.py +++ b/test_regress/t/t_lint_implicit_def_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only -Wall -Wno-DECLFILENAME"], +test.lint(verilator_flags2=["-Wall -Wno-DECLFILENAME"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_lint_implicit_func_bad.py b/test_regress/t/t_lint_implicit_func_bad.py index c0608225e..0db29c160 100755 --- a/test_regress/t/t_lint_implicit_func_bad.py +++ b/test_regress/t/t_lint_implicit_func_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only -Wall -Wno-DECLFILENAME"], +test.lint(verilator_flags2=["-Wall -Wno-DECLFILENAME"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_lint_implicit_type_bad.py b/test_regress/t/t_lint_implicit_type_bad.py index ff290aea9..383be3ece 100755 --- a/test_regress/t/t_lint_implicit_type_bad.py +++ b/test_regress/t/t_lint_implicit_type_bad.py @@ -12,7 +12,7 @@ import vltest_bootstrap test.scenarios('vlt') # --debug-check adds extra internal message, otherwise golden log would vary -test.lint(verilator_flags2=["--lint-only --no-debug-check -Wall -Wno-DECLFILENAME"], +test.lint(verilator_flags2=["--no-debug-check -Wall -Wno-DECLFILENAME"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_lint_import_name2_bad.py b/test_regress/t/t_lint_import_name2_bad.py index c0608225e..0db29c160 100755 --- a/test_regress/t/t_lint_import_name2_bad.py +++ b/test_regress/t/t_lint_import_name2_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only -Wall -Wno-DECLFILENAME"], +test.lint(verilator_flags2=["-Wall -Wno-DECLFILENAME"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_lint_import_name_bad.py b/test_regress/t/t_lint_import_name_bad.py index c0608225e..0db29c160 100755 --- a/test_regress/t/t_lint_import_name_bad.py +++ b/test_regress/t/t_lint_import_name_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only -Wall -Wno-DECLFILENAME"], +test.lint(verilator_flags2=["-Wall -Wno-DECLFILENAME"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_lint_importstar_bad.py b/test_regress/t/t_lint_importstar_bad.py index c0608225e..0db29c160 100755 --- a/test_regress/t/t_lint_importstar_bad.py +++ b/test_regress/t/t_lint_importstar_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only -Wall -Wno-DECLFILENAME"], +test.lint(verilator_flags2=["-Wall -Wno-DECLFILENAME"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_lint_incabspath_bad.py b/test_regress/t/t_lint_incabspath_bad.py index a825eadf4..bed07adeb 100755 --- a/test_regress/t/t_lint_incabspath_bad.py +++ b/test_regress/t/t_lint_incabspath_bad.py @@ -12,7 +12,7 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = "t/t_lint_incabspath.v" -test.lint(verilator_flags2=["--lint-only -Wall -Wno-DECLFILENAME"], +test.lint(verilator_flags2=["-Wall -Wno-DECLFILENAME"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_lint_latch_bad.py b/test_regress/t/t_lint_latch_bad.py index 8441aff14..d3baf9587 100755 --- a/test_regress/t/t_lint_latch_bad.py +++ b/test_regress/t/t_lint_latch_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only -Wwarn-style -Wno-DECLFILENAME"], +test.lint(verilator_flags2=["-Wwarn-style -Wno-DECLFILENAME"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_lint_lint_bad.py b/test_regress/t/t_lint_lint_bad.py index c0e512c32..a34a333d4 100755 --- a/test_regress/t/t_lint_lint_bad.py +++ b/test_regress/t/t_lint_lint_bad.py @@ -11,8 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wwarn-lint"], - fails=True, - expect_filename=test.golden_filename) +test.lint(verilator_flags2=["-Wwarn-lint"], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_lint_lint_no.py b/test_regress/t/t_lint_lint_no.py index e4a6ba878..1a4025a8e 100755 --- a/test_regress/t/t_lint_lint_no.py +++ b/test_regress/t/t_lint_lint_no.py @@ -12,6 +12,6 @@ import vltest_bootstrap test.scenarios('linter') test.top_filename = 't/t_lint_lint_bad.v' -test.lint(verilator_flags2=["--lint-only -Wwarn-lint -Wno-lint"]) +test.lint(verilator_flags2=["-Wwarn-lint -Wno-lint"]) test.passes() diff --git a/test_regress/t/t_lint_misindent_bad.py b/test_regress/t/t_lint_misindent_bad.py index f05a78df6..946e0710c 100755 --- a/test_regress/t/t_lint_misindent_bad.py +++ b/test_regress/t/t_lint_misindent_bad.py @@ -12,7 +12,7 @@ import vltest_bootstrap test.scenarios('vlt') test.lint(fails=True, - verilator_flags2=["--lint-only -Wall -Wno-DECLFILENAME"], + verilator_flags2=["-Wall -Wno-DECLFILENAME"], expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_lint_mod_paren_bad.py b/test_regress/t/t_lint_mod_paren_bad.py index 3ce9351b2..215c97d4b 100755 --- a/test_regress/t/t_lint_mod_paren_bad.py +++ b/test_regress/t/t_lint_mod_paren_bad.py @@ -11,8 +11,6 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only -Wno-DECLFILENAME"], - fails=True, - expect_filename=test.golden_filename) +test.lint(verilator_flags2=["-Wno-DECLFILENAME"], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_lint_modport_dir_bad.py b/test_regress/t/t_lint_modport_dir_bad.py index 53e81eb85..ae0961bea 100755 --- a/test_regress/t/t_lint_modport_dir_bad.py +++ b/test_regress/t/t_lint_modport_dir_bad.py @@ -11,8 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wno-DECLFILENAME"], - fails=True, - expect_filename=test.golden_filename) +test.lint(verilator_flags2=["-Wno-DECLFILENAME"], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_lint_once_bad.py b/test_regress/t/t_lint_once_bad.py index 9690ccd5f..fcc71dff4 100755 --- a/test_regress/t/t_lint_once_bad.py +++ b/test_regress/t/t_lint_once_bad.py @@ -11,8 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wwarn-UNUSED"], - fails=True, - expect_filename=test.golden_filename) +test.lint(verilator_flags2=["-Wwarn-UNUSED"], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_lint_pragma_protected.py b/test_regress/t/t_lint_pragma_protected.py index e0fa30056..3b4ab2b45 100755 --- a/test_regress/t/t_lint_pragma_protected.py +++ b/test_regress/t/t_lint_pragma_protected.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=['--lint-only -Wno-PROTECTED']) +test.lint(verilator_flags2=['-Wno-PROTECTED']) test.passes() diff --git a/test_regress/t/t_lint_pragma_protected_bad.py b/test_regress/t/t_lint_pragma_protected_bad.py index dde27733d..e1346660f 100755 --- a/test_regress/t/t_lint_pragma_protected_bad.py +++ b/test_regress/t/t_lint_pragma_protected_bad.py @@ -11,8 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(fails=True, - verilator_flags2=['--lint-only -Wpedantic'], - expect_filename=test.golden_filename) +test.lint(fails=True, verilator_flags2=['-Wpedantic'], expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_lint_realcvt_bad.py b/test_regress/t/t_lint_realcvt_bad.py index acbdae169..f1cb99e8a 100755 --- a/test_regress/t/t_lint_realcvt_bad.py +++ b/test_regress/t/t_lint_realcvt_bad.py @@ -11,8 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wwarn-REALCVT"], - fails=True, - expect_filename=test.golden_filename) +test.lint(verilator_flags2=["-Wwarn-REALCVT"], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_lint_setout_bad_noinl.py b/test_regress/t/t_lint_setout_bad_noinl.py index c10fcd5f7..829e79ef1 100755 --- a/test_regress/t/t_lint_setout_bad_noinl.py +++ b/test_regress/t/t_lint_setout_bad_noinl.py @@ -12,8 +12,6 @@ import vltest_bootstrap test.scenarios('linter') test.top_filename = "t/t_lint_setout_bad.v" -test.lint(verilator_flags2=["--lint-only -fno-inline"], - fails=True, - expect_filename=test.golden_filename) +test.lint(verilator_flags2=["-fno-inline"], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_lint_style_bad.py b/test_regress/t/t_lint_style_bad.py index 12a1a5ee2..308d8264d 100755 --- a/test_regress/t/t_lint_style_bad.py +++ b/test_regress/t/t_lint_style_bad.py @@ -11,8 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wwarn-style"], - fails=True, - expect_filename=test.golden_filename) +test.lint(verilator_flags2=["-Wwarn-style"], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_lint_style_no.py b/test_regress/t/t_lint_style_no.py index dd5d56a49..e53007ac0 100755 --- a/test_regress/t/t_lint_style_no.py +++ b/test_regress/t/t_lint_style_no.py @@ -12,6 +12,6 @@ import vltest_bootstrap test.scenarios('linter') test.top_filename = 't/t_lint_style_bad.v' -test.lint(verilator_flags2=["--lint-only -Wwarn-style -Wno-style"]) +test.lint(verilator_flags2=["-Wwarn-style -Wno-style"]) test.passes() diff --git a/test_regress/t/t_lint_syncasyncnet_bad.py b/test_regress/t/t_lint_syncasyncnet_bad.py index 44eef5de9..3753ea9a5 100755 --- a/test_regress/t/t_lint_syncasyncnet_bad.py +++ b/test_regress/t/t_lint_syncasyncnet_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wall -Wno-DECLFILENAME --if-depth 10"], +test.lint(verilator_flags2=["-Wall -Wno-DECLFILENAME --if-depth 10"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_lint_unsup_deassign.py b/test_regress/t/t_lint_unsup_deassign.py index ef40a09c4..4dd72181f 100755 --- a/test_regress/t/t_lint_unsup_deassign.py +++ b/test_regress/t/t_lint_unsup_deassign.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only --bbox-unsup"]) +test.lint(verilator_flags2=["--bbox-unsup"]) test.passes() diff --git a/test_regress/t/t_lint_unsup_mixed.py b/test_regress/t/t_lint_unsup_mixed.py index ef40a09c4..4dd72181f 100755 --- a/test_regress/t/t_lint_unsup_mixed.py +++ b/test_regress/t/t_lint_unsup_mixed.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only --bbox-unsup"]) +test.lint(verilator_flags2=["--bbox-unsup"]) test.passes() diff --git a/test_regress/t/t_lint_unused.py b/test_regress/t/t_lint_unused.py index 3993e1fec..9e9a93a06 100755 --- a/test_regress/t/t_lint_unused.py +++ b/test_regress/t/t_lint_unused.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only --bbox-sys --bbox-unsup -Wall -Wno-DECLFILENAME"]) +test.lint(verilator_flags2=["--bbox-sys --bbox-unsup -Wall -Wno-DECLFILENAME"]) test.passes() diff --git a/test_regress/t/t_lint_unused_iface.py b/test_regress/t/t_lint_unused_iface.py index 8bb4f3a66..dabf243d0 100755 --- a/test_regress/t/t_lint_unused_iface.py +++ b/test_regress/t/t_lint_unused_iface.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only -Wall -Wno-DECLFILENAME"]) +test.lint(verilator_flags2=["-Wall -Wno-DECLFILENAME"]) test.passes() diff --git a/test_regress/t/t_lint_unused_iface_bad.py b/test_regress/t/t_lint_unused_iface_bad.py index c6b6f1747..ad1a5882d 100755 --- a/test_regress/t/t_lint_unused_iface_bad.py +++ b/test_regress/t/t_lint_unused_iface_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wall -Wno-DECLFILENAME"], +test.lint(verilator_flags2=["-Wall -Wno-DECLFILENAME"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_lint_unused_tri.py b/test_regress/t/t_lint_unused_tri.py index 8bb4f3a66..dabf243d0 100755 --- a/test_regress/t/t_lint_unused_tri.py +++ b/test_regress/t/t_lint_unused_tri.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only -Wall -Wno-DECLFILENAME"]) +test.lint(verilator_flags2=["-Wall -Wno-DECLFILENAME"]) test.passes() diff --git a/test_regress/t/t_lint_widthexpand_docs_bad.py b/test_regress/t/t_lint_widthexpand_docs_bad.py index 4e7afe03b..3c604cd18 100755 --- a/test_regress/t/t_lint_widthexpand_docs_bad.py +++ b/test_regress/t/t_lint_widthexpand_docs_bad.py @@ -11,9 +11,7 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only"], - fails=test.vlt_all, - expect_filename=test.golden_filename) +test.lint(fails=test.vlt_all, expect_filename=test.golden_filename) test.extract(in_filename=test.top_filename, out_filename=test.root + "/docs/gen/ex_WIDTHEXPAND_1_faulty.rst", diff --git a/test_regress/t/t_no_std_bad.py b/test_regress/t/t_no_std_bad.py index e186f97ef..b0857dbf5 100755 --- a/test_regress/t/t_no_std_bad.py +++ b/test_regress/t/t_no_std_bad.py @@ -11,8 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(fails=True, - verilator_flags2=["--no-std", "--binary -Wall"], - expect_filename=test.golden_filename) +test.lint(fails=True, verilator_flags2=["--no-std", "-Wall"], expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_no_std_pkg_bad.py b/test_regress/t/t_no_std_pkg_bad.py index efc541e29..d8c88252c 100755 --- a/test_regress/t/t_no_std_pkg_bad.py +++ b/test_regress/t/t_no_std_pkg_bad.py @@ -14,7 +14,7 @@ test.top_filename = "t/t_no_std_bad.v" test.golden_filename = "t/t_no_std_bad.out" test.lint(fails=True, - verilator_flags2=["--no-std-package", "--binary -Wall"], + verilator_flags2=["--no-std-package", "-Wall"], expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_order_blkloopinit_bad.py b/test_regress/t/t_order_blkloopinit_bad.py index 7009ab7ce..e33e10acf 100755 --- a/test_regress/t/t_order_blkloopinit_bad.py +++ b/test_regress/t/t_order_blkloopinit_bad.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=[], fails=True, expect_filename=test.golden_filename) +test.lint(fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_param_default_presv_bad.py b/test_regress/t/t_param_default_presv_bad.py index a99a63850..4730f0b37 100755 --- a/test_regress/t/t_param_default_presv_bad.py +++ b/test_regress/t/t_param_default_presv_bad.py @@ -12,7 +12,7 @@ import vltest_bootstrap test.scenarios('linter') test.top_filename = "t/t_param_default_bad.v" -test.lint(verilator_flags2=["--lint-only --language 1800-2005"], +test.lint(verilator_flags2=["--language 1800-2005"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_param_sel_range.py b/test_regress/t/t_param_sel_range.py index 905b4884b..282910f54 100755 --- a/test_regress/t/t_param_sel_range.py +++ b/test_regress/t/t_param_sel_range.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wno-SELRANGE"]) +test.lint(verilator_flags2=["-Wno-SELRANGE"]) test.passes() diff --git a/test_regress/t/t_pipe_exit_bad.py b/test_regress/t/t_pipe_exit_bad.py index 3d64d816b..7e0fef5d9 100755 --- a/test_regress/t/t_pipe_exit_bad.py +++ b/test_regress/t/t_pipe_exit_bad.py @@ -12,7 +12,8 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = "t/t_pipe_filter.v" -test.lint(verilator_flags2=['-E --pipe-filter \'python3 t/t_pipe_exit_bad_pf.pf\' '], fails=True) +test.compile(verilator_flags2=['-E --pipe-filter \'python3 t/t_pipe_exit_bad_pf.pf\' '], + fails=True) test.file_grep(test.compile_log_filename, r'%Error: t_pipe_exit_bad_pf.pf: Intentional bad exit status....*') diff --git a/test_regress/t/t_pp_pragma_bad.py b/test_regress/t/t_pp_pragma_bad.py index f570493aa..e9ac373c2 100755 --- a/test_regress/t/t_pp_pragma_bad.py +++ b/test_regress/t/t_pp_pragma_bad.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["-E -Wpedantic"], fails=True, expect_filename=test.golden_filename) +test.compile(verilator_flags2=["-E -Wpedantic"], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_process_bad.py b/test_regress/t/t_process_bad.py index 786f00c3e..9536f58b9 100755 --- a/test_regress/t/t_process_bad.py +++ b/test_regress/t/t_process_bad.py @@ -11,8 +11,8 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--json-only", "--timing"], - fails=True, - expect_filename=test.golden_filename) +test.compile(verilator_flags2=["--json-only", "--timing"], + fails=True, + expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_randomize_complex_member_bad.py b/test_regress/t/t_randomize_complex_member_bad.py index 830a90560..a551e844d 100755 --- a/test_regress/t/t_randomize_complex_member_bad.py +++ b/test_regress/t/t_randomize_complex_member_bad.py @@ -14,6 +14,6 @@ test.scenarios('linter') if not test.have_solver: test.skip("No constraint solver installed") -test.lint(verilator_flags2=["--lint-only"], fails=True, expect_filename=test.golden_filename) +test.lint(fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_sarif.sarif.out b/test_regress/t/t_sarif.sarif.out index 94b260d4d..61eb1a6ea 100644 --- a/test_regress/t/t_sarif.sarif.out +++ b/test_regress/t/t_sarif.sarif.out @@ -26,7 +26,7 @@ }, "invocations": [ { - "commandLine": "--prefix Vt_sarif -cc -Mdir obj_vlt/t_sarif --fdedup --debug-check --comp-limit-members 10 --x-assign unique -Wno-fatal --diagnostics-sarif --no-skip-identical -f input.vc +define+TEST_OBJ_DIR=obj_vlt/t_sarif +define+TEST_DUMPFILE=obj_vlt/t_sarif/simx.vcd t/t_sarif.v +librescan +notimingchecks +libext+.v -y t +incdir+t", + "commandLine": "--prefix Vt_sarif --lint-only -Mdir obj_vlt/t_sarif --debug-check --x-assign unique -Wno-fatal --diagnostics-sarif --no-skip-identical -f input.vc +define+TEST_OBJ_DIR=obj_vlt/t_sarif +define+TEST_DUMPFILE=obj_vlt/t_sarif/simx.vcd t/t_sarif.v +librescan +notimingchecks +libext+.v -y t +incdir+t", "executionSuccessful": true } ], diff --git a/test_regress/t/t_std_process_self.py b/test_regress/t/t_std_process_self.py index 401e3d56e..1211b1f6f 100755 --- a/test_regress/t/t_std_process_self.py +++ b/test_regress/t/t_std_process_self.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('simulator') -test.lint(verilator_flags2=["--binary"]) +test.compile(verilator_flags2=["--timing"]) test.passes() diff --git a/test_regress/t/t_std_process_self_std.py b/test_regress/t/t_std_process_self_std.py index ca3d1c05c..15da65e59 100755 --- a/test_regress/t/t_std_process_self_std.py +++ b/test_regress/t/t_std_process_self_std.py @@ -12,6 +12,6 @@ import vltest_bootstrap test.scenarios('simulator') test.top_filename = "t/t_std_process_self.v" -test.lint(verilator_flags2=["--binary --DUSE_STD_PREFIX"]) +test.compile(verilator_flags2=["--binary --DUSE_STD_PREFIX"]) test.passes() diff --git a/test_regress/t/t_std_randomize_mod.py b/test_regress/t/t_std_randomize_mod.py index 6815786ce..7dcbd6ae6 100755 --- a/test_regress/t/t_std_randomize_mod.py +++ b/test_regress/t/t_std_randomize_mod.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('simulator') -test.lint(verilator_flags2=["--binary"]) +test.lint() test.passes() diff --git a/test_regress/t/t_std_randomize_no_args.py b/test_regress/t/t_std_randomize_no_args.py index 6815786ce..7dcbd6ae6 100755 --- a/test_regress/t/t_std_randomize_no_args.py +++ b/test_regress/t/t_std_randomize_no_args.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('simulator') -test.lint(verilator_flags2=["--binary"]) +test.lint() test.passes() diff --git a/test_regress/t/t_timescale_lint.py b/test_regress/t/t_timescale_lint.py index 5e513aa7c..eb4fffae6 100755 --- a/test_regress/t/t_timescale_lint.py +++ b/test_regress/t/t_timescale_lint.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only --timescale-override 1ns/1ns"]) +test.lint(verilator_flags2=["--timescale-override 1ns/1ns"]) test.passes() diff --git a/test_regress/t/t_timescale_lint2.py b/test_regress/t/t_timescale_lint2.py index 6e393030b..e8ac2b25f 100755 --- a/test_regress/t/t_timescale_lint2.py +++ b/test_regress/t/t_timescale_lint2.py @@ -12,6 +12,6 @@ import vltest_bootstrap test.scenarios('linter') test.top_filename = "t/t_timescale_lint.v" -test.lint(verilator_flags2=["--lint-only --timescale 1ns/1ns"]) +test.lint(verilator_flags2=["--timescale 1ns/1ns"]) test.passes() diff --git a/test_regress/t/t_timescale_lint_bad.py b/test_regress/t/t_timescale_lint_bad.py index 8842c380b..3d6a0cca6 100755 --- a/test_regress/t/t_timescale_lint_bad.py +++ b/test_regress/t/t_timescale_lint_bad.py @@ -12,6 +12,6 @@ import vltest_bootstrap test.scenarios('linter') test.top_filename = "t/t_timescale_lint.v" -test.lint(verilator_flags2=["--lint-only"], fails=True, expect_filename=test.golden_filename) +test.lint(fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_trace_multi_bad.py b/test_regress/t/t_trace_multi_bad.py index 4d6bcd9ce..cbb233614 100755 --- a/test_regress/t/t_trace_multi_bad.py +++ b/test_regress/t/t_trace_multi_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags=['--trace-fst --trace-vcd'], +test.lint(verilator_flags2=['--trace-fst --trace-vcd'], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_typedef_no_bad.py b/test_regress/t/t_typedef_no_bad.py index 33cb398b7..e33e10acf 100755 --- a/test_regress/t/t_typedef_no_bad.py +++ b/test_regress/t/t_typedef_no_bad.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--json-only"], fails=True, expect_filename=test.golden_filename) +test.lint(fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_udp_bad.py b/test_regress/t/t_udp_bad.py index a0f699e19..d0a97a711 100755 --- a/test_regress/t/t_udp_bad.py +++ b/test_regress/t/t_udp_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only --bbox-unsup"], +test.lint(verilator_flags2=["--bbox-unsup"], fails=test.vlt_all, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_udp_tableeof_bad.py b/test_regress/t/t_udp_tableeof_bad.py index 6b41e2961..12b1678d4 100755 --- a/test_regress/t/t_udp_tableeof_bad.py +++ b/test_regress/t/t_udp_tableeof_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only --bbox-unsup"], fails=test.vlt_all) +test.lint(verilator_flags2=["--bbox-unsup"], fails=test.vlt_all) # Cannot use .out, get "$end" or "end of file" depending on bison version test.file_grep(test.compile_log_filename, r"EOF in 'table'") diff --git a/test_regress/t/t_var_bad_hide.py b/test_regress/t/t_var_bad_hide.py index 2076da382..84ba24c85 100755 --- a/test_regress/t/t_var_bad_hide.py +++ b/test_regress/t/t_var_bad_hide.py @@ -11,8 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wwarn-VARHIDDEN"], - fails=True, - expect_filename=test.golden_filename) +test.lint(verilator_flags2=["-Wwarn-VARHIDDEN"], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_var_bad_hide2.py b/test_regress/t/t_var_bad_hide2.py index d546b464e..04d516c0f 100755 --- a/test_regress/t/t_var_bad_hide2.py +++ b/test_regress/t/t_var_bad_hide2.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wwarn-VARHIDDEN"], +test.lint(verilator_flags2=["-Wwarn-VARHIDDEN"], fails=test.vlt_all, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_var_bad_hide_docs.py b/test_regress/t/t_var_bad_hide_docs.py index 25caf8912..1b7e7a797 100755 --- a/test_regress/t/t_var_bad_hide_docs.py +++ b/test_regress/t/t_var_bad_hide_docs.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=["--lint-only -Wwarn-VARHIDDEN"], +test.lint(verilator_flags2=["-Wwarn-VARHIDDEN"], fails=test.vlt_all, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_var_in_assign_bad.py b/test_regress/t/t_var_in_assign_bad.py index b7ddcf7bd..5629f9846 100755 --- a/test_regress/t/t_var_in_assign_bad.py +++ b/test_regress/t/t_var_in_assign_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only --Mdir obj_lint_only"], +test.lint(verilator_flags2=["--Mdir obj_lint_only"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_varref_scope_in_interface.py b/test_regress/t/t_varref_scope_in_interface.py index fbe7b466b..4cd2b6790 100755 --- a/test_regress/t/t_varref_scope_in_interface.py +++ b/test_regress/t/t_varref_scope_in_interface.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only", "--timing"]) +test.lint(verilator_flags2=["--timing"]) test.passes() diff --git a/test_regress/t/t_virtual_interface_method_bad.py b/test_regress/t/t_virtual_interface_method_bad.py index 54f436672..2ce66844f 100755 --- a/test_regress/t/t_virtual_interface_method_bad.py +++ b/test_regress/t/t_virtual_interface_method_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only --language 1800-2017"], +test.lint(verilator_flags2=["--language 1800-2017"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_vlt_match_contents.py b/test_regress/t/t_vlt_match_contents.py index 3c4174c36..1c0bcea9a 100755 --- a/test_regress/t/t_vlt_match_contents.py +++ b/test_regress/t/t_vlt_match_contents.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only -Wall t/t_vlt_match_contents.vlt"], +test.lint(verilator_flags2=["-Wall t/t_vlt_match_contents.vlt"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_vlt_match_error_1.py b/test_regress/t/t_vlt_match_error_1.py index 14c18fda5..209cca7be 100755 --- a/test_regress/t/t_vlt_match_error_1.py +++ b/test_regress/t/t_vlt_match_error_1.py @@ -12,10 +12,9 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = "t/t_vlt_match_error.v" -test.lint(verilator_flags2=[ - "-DT_VLT_MATCH_ERROR_1 --lint-only -Wall t/t_vlt_match_error.v t/t_vlt_match_error.vlt" -], - fails=True, - expect_filename=test.golden_filename) +test.lint( + verilator_flags2=["-DT_VLT_MATCH_ERROR_1 -Wall t/t_vlt_match_error.v t/t_vlt_match_error.vlt"], + fails=True, + expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_vlt_match_error_2.py b/test_regress/t/t_vlt_match_error_2.py index 98c8b2327..3d67fbdd1 100755 --- a/test_regress/t/t_vlt_match_error_2.py +++ b/test_regress/t/t_vlt_match_error_2.py @@ -12,10 +12,9 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = "t/t_vlt_match_error.v" -test.lint(verilator_flags2=[ - "-DT_VLT_MATCH_ERROR_2 --lint-only -Wall t/t_vlt_match_error.v t/t_vlt_match_error.vlt" -], - fails=True, - expect_filename=test.golden_filename) +test.lint( + verilator_flags2=["-DT_VLT_MATCH_ERROR_2 -Wall t/t_vlt_match_error.v t/t_vlt_match_error.vlt"], + fails=True, + expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_vlt_match_error_3.py b/test_regress/t/t_vlt_match_error_3.py index 53a1dae32..423cb1d88 100755 --- a/test_regress/t/t_vlt_match_error_3.py +++ b/test_regress/t/t_vlt_match_error_3.py @@ -12,10 +12,9 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = "t/t_vlt_match_error.v" -test.lint(verilator_flags2=[ - "-DT_VLT_MATCH_ERROR_3 --lint-only -Wall t/t_vlt_match_error.v t/t_vlt_match_error.vlt" -], - fails=True, - expect_filename=test.golden_filename) +test.lint( + verilator_flags2=["-DT_VLT_MATCH_ERROR_3 -Wall t/t_vlt_match_error.v t/t_vlt_match_error.vlt"], + fails=True, + expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_vlt_warn.py b/test_regress/t/t_vlt_warn.py index 58ef49915..4260c4bdc 100755 --- a/test_regress/t/t_vlt_warn.py +++ b/test_regress/t/t_vlt_warn.py @@ -11,6 +11,6 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only -Wall t/t_vlt_warn.vlt"]) +test.lint(verilator_flags2=["-Wall t/t_vlt_warn.vlt"]) test.passes() diff --git a/test_regress/t/t_vlt_warn_bad.py b/test_regress/t/t_vlt_warn_bad.py index b55f59369..b06923482 100755 --- a/test_regress/t/t_vlt_warn_bad.py +++ b/test_regress/t/t_vlt_warn_bad.py @@ -12,7 +12,7 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = "t/t_vlt_warn.v" -test.lint(verilator_flags2=["--lint-only t/t_vlt_warn_bad.vlt"], +test.lint(verilator_flags2=["t/t_vlt_warn_bad.vlt"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_vlt_warn_ecode_bad.py b/test_regress/t/t_vlt_warn_ecode_bad.py index 28f9b6683..b89538328 100755 --- a/test_regress/t/t_vlt_warn_ecode_bad.py +++ b/test_regress/t/t_vlt_warn_ecode_bad.py @@ -12,7 +12,7 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = "t/t_vlt_warn.v" -test.lint(verilator_flags2=["--lint-only t/t_vlt_warn_ecode_bad.vlt"], +test.lint(verilator_flags2=["t/t_vlt_warn_ecode_bad.vlt"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_wait.py b/test_regress/t/t_wait.py index c2eb14c07..8fe17ddf9 100755 --- a/test_regress/t/t_wait.py +++ b/test_regress/t/t_wait.py @@ -11,8 +11,6 @@ import vltest_bootstrap test.scenarios('linter') -test.lint(verilator_flags2=['--lint-only --no-timing'], - fails=True, - expect_filename=test.golden_filename) +test.lint(verilator_flags2=['--no-timing'], fails=True, expect_filename=test.golden_filename) test.passes() diff --git a/test_regress/t/t_wire_beh1364_bad.py b/test_regress/t/t_wire_beh1364_bad.py index 9e2f7e267..797770aeb 100755 --- a/test_regress/t/t_wire_beh1364_bad.py +++ b/test_regress/t/t_wire_beh1364_bad.py @@ -13,7 +13,7 @@ test.scenarios('vlt') waiver_filename = test.obj_dir + "/" + test.name + "_waiver.vlt" -test.lint(verilator_flags2=["--lint-only --language 1364-2001 --waiver-output", waiver_filename], +test.lint(verilator_flags2=["--language 1364-2001 --waiver-output", waiver_filename], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_wire_beh1800_bad.py b/test_regress/t/t_wire_beh1800_bad.py index 54f436672..2ce66844f 100755 --- a/test_regress/t/t_wire_beh1800_bad.py +++ b/test_regress/t/t_wire_beh1800_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only --language 1800-2017"], +test.lint(verilator_flags2=["--language 1800-2017"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_wire_behp1364_bad.py b/test_regress/t/t_wire_behp1364_bad.py index 773f8b8e1..c3c017906 100755 --- a/test_regress/t/t_wire_behp1364_bad.py +++ b/test_regress/t/t_wire_behp1364_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only --language 1364-2001"], +test.lint(verilator_flags2=["--language 1364-2001"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_wire_behp1800_bad.py b/test_regress/t/t_wire_behp1800_bad.py index 54f436672..2ce66844f 100755 --- a/test_regress/t/t_wire_behp1800_bad.py +++ b/test_regress/t/t_wire_behp1800_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only --language 1800-2017"], +test.lint(verilator_flags2=["--language 1800-2017"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_wire_self_bad.py b/test_regress/t/t_wire_self_bad.py index 54f436672..2ce66844f 100755 --- a/test_regress/t/t_wire_self_bad.py +++ b/test_regress/t/t_wire_self_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only --language 1800-2017"], +test.lint(verilator_flags2=["--language 1800-2017"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_with_suggest_bad.py b/test_regress/t/t_with_suggest_bad.py index 54f436672..2ce66844f 100755 --- a/test_regress/t/t_with_suggest_bad.py +++ b/test_regress/t/t_with_suggest_bad.py @@ -11,7 +11,7 @@ import vltest_bootstrap test.scenarios('vlt') -test.lint(verilator_flags2=["--lint-only --language 1800-2017"], +test.lint(verilator_flags2=["--language 1800-2017"], fails=True, expect_filename=test.golden_filename) diff --git a/test_regress/t/t_xml_deprecated_bad.py b/test_regress/t/t_xml_deprecated_bad.py index 7fcdf4d0a..7d78f956d 100755 --- a/test_regress/t/t_xml_deprecated_bad.py +++ b/test_regress/t/t_xml_deprecated_bad.py @@ -12,8 +12,8 @@ import vltest_bootstrap test.scenarios('vlt') test.top_filename = 't/t_EXAMPLE.v' -test.lint(verilator_flags2=["--xml-only --xml-output /dev/null"], - fails=True, - expect_filename=test.golden_filename) +test.compile(verilator_flags2=["--xml-only --xml-output /dev/null"], + fails=True, + expect_filename=test.golden_filename) test.passes()