From d595d4c7ac581e4dde4b1a1a5b0fb46c97f97c03 Mon Sep 17 00:00:00 2001
From: Wilson Snyder <wsnyder@wsnyder.org>
Date: Sat, 20 Jan 2024 10:57:00 -0500
Subject: [PATCH] Tests: Use --binary with some --timing tests

---
 test_regress/t/t_altera_lpm_counter.pl          | 2 +-
 test_regress/t/t_delay_incr.pl                  | 2 +-
 test_regress/t/t_delay_incr_timing.pl           | 2 +-
 test_regress/t/t_fork_join_none_any_nested.pl   | 2 +-
 test_regress/t/t_fork_label.pl                  | 2 +-
 test_regress/t/t_fork_timing.pl                 | 2 +-
 test_regress/t/t_func_lib_sub_timing.pl         | 2 +-
 test_regress/t/t_interconnect.pl                | 2 +-
 test_regress/t/t_mailbox_class.pl               | 2 +-
 test_regress/t/t_math_signed5.pl                | 2 +-
 test_regress/t/t_math_signed5_timing.pl         | 2 +-
 test_regress/t/t_package_ddecl.pl               | 2 +-
 test_regress/t/t_package_ddecl_timing.pl        | 2 +-
 test_regress/t/t_parse_delay.pl                 | 2 +-
 test_regress/t/t_parse_delay_timing.pl          | 2 +-
 test_regress/t/t_process.pl                     | 2 +-
 test_regress/t/t_process_fork.pl                | 2 +-
 test_regress/t/t_process_rand.pl                | 2 +-
 test_regress/t/t_process_std.pl                 | 2 +-
 test_regress/t/t_queue_persistence_inl_unsup.pl | 2 +-
 test_regress/t/t_queue_persistence_noinl.pl     | 2 +-
 test_regress/t/t_selextract_in_paramextends.pl  | 2 +-
 test_regress/t/t_semaphore_class.pl             | 2 +-
 test_regress/t/t_suspendable_deep.pl            | 2 +-
 test_regress/t/t_uvm_all.pl                     | 2 +-
 test_regress/t/t_var_in_fork.pl                 | 2 +-
 test_regress/t/t_wait_order.pl                  | 2 +-
 test_regress/t/t_wait_timing.pl                 | 2 +-
 test_regress/t/t_while_timing_control.pl        | 2 +-
 29 files changed, 29 insertions(+), 29 deletions(-)

diff --git a/test_regress/t/t_altera_lpm_counter.pl b/test_regress/t/t_altera_lpm_counter.pl
index 78ec8c557..3f4aa0eed 100755
--- a/test_regress/t/t_altera_lpm_counter.pl
+++ b/test_regress/t/t_altera_lpm_counter.pl
@@ -14,7 +14,7 @@ top_filename("t/t_altera_lpm.v");
 (my $module = $Self->{name}) =~ s/.*t_altera_//;
 
 compile(
-    verilator_flags2 => ["--top-module ${module}", "--no-timing"]
+    verilator_flags2 => ["--top-module ${module}", "--binary --no-timing"]
     );
 
 ok(1);
diff --git a/test_regress/t/t_delay_incr.pl b/test_regress/t/t_delay_incr.pl
index 3e8a3c919..ff38a8470 100755
--- a/test_regress/t/t_delay_incr.pl
+++ b/test_regress/t/t_delay_incr.pl
@@ -11,7 +11,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 scenarios(simulator => 1);
 
 compile(
-    verilator_flags2 => ['-Wno-STMTDLY -Wno-ASSIGNDLY --no-timing'],
+    verilator_flags2 => ['--binary --no-timing -Wno-STMTDLY -Wno-ASSIGNDLY'],
     );
 
 execute(
diff --git a/test_regress/t/t_delay_incr_timing.pl b/test_regress/t/t_delay_incr_timing.pl
index 7bc99d468..d60b1c8bd 100755
--- a/test_regress/t/t_delay_incr_timing.pl
+++ b/test_regress/t/t_delay_incr_timing.pl
@@ -16,7 +16,7 @@ top_filename("t/t_delay_incr.v");
 
 compile(
     timing_loop => 1,
-    verilator_flags2 => ['--timing -Wno-ZERODLY'],
+    verilator_flags2 => ['--binary --timing -Wno-ZERODLY'],
     );
 
 execute(
diff --git a/test_regress/t/t_fork_join_none_any_nested.pl b/test_regress/t/t_fork_join_none_any_nested.pl
index b267631e9..fec4c2942 100755
--- a/test_regress/t/t_fork_join_none_any_nested.pl
+++ b/test_regress/t/t_fork_join_none_any_nested.pl
@@ -11,7 +11,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 scenarios(simulator => 1);
 
 compile(
-    verilator_flags2 => ["--timing"],
+    verilator_flags2 => ["--binary --timing"],
     );
 
 execute(
diff --git a/test_regress/t/t_fork_label.pl b/test_regress/t/t_fork_label.pl
index 8bb4480e1..dc3cfd898 100755
--- a/test_regress/t/t_fork_label.pl
+++ b/test_regress/t/t_fork_label.pl
@@ -11,7 +11,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 scenarios(simulator => 1);
 
 compile(
-    verilator_flags2 => ['--no-timing'],
+    verilator_flags2 => ['--binary --no-timing'],
     );
 
 execute();
diff --git a/test_regress/t/t_fork_timing.pl b/test_regress/t/t_fork_timing.pl
index db741a7d5..b62f5837f 100755
--- a/test_regress/t/t_fork_timing.pl
+++ b/test_regress/t/t_fork_timing.pl
@@ -13,7 +13,7 @@ scenarios(vlt => 1);
 top_filename("t/t_fork.v");
 
 compile(
-    verilator_flags2 => ["--timing"],
+    verilator_flags2 => ["--binary --timing"],
     );
 
 execute(
diff --git a/test_regress/t/t_func_lib_sub_timing.pl b/test_regress/t/t_func_lib_sub_timing.pl
index 7498c6694..6101d0fa0 100755
--- a/test_regress/t/t_func_lib_sub_timing.pl
+++ b/test_regress/t/t_func_lib_sub_timing.pl
@@ -13,7 +13,7 @@ scenarios(vlt => 1);  # UNOPTTHREADS in vltmt
 top_filename("t/t_func_lib_sub.v");
 
 compile(
-    verilator_flags2 => ["--timing"],
+    verilator_flags2 => ["--binary --timing"],
     );
 
 # No execute
diff --git a/test_regress/t/t_interconnect.pl b/test_regress/t/t_interconnect.pl
index 2a516857f..7cb28e364 100755
--- a/test_regress/t/t_interconnect.pl
+++ b/test_regress/t/t_interconnect.pl
@@ -11,7 +11,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 scenarios(simulator => 1);
 
 compile(
-    verilator_flags2 => ["--timing"],
+    verilator_flags2 => ["--binary --timing"],
     fails => $Self->{vlt_all},
     expect_filename => $Self->{golden_filename},
     );
diff --git a/test_regress/t/t_mailbox_class.pl b/test_regress/t/t_mailbox_class.pl
index 6e30bd25d..91589388d 100755
--- a/test_regress/t/t_mailbox_class.pl
+++ b/test_regress/t/t_mailbox_class.pl
@@ -11,7 +11,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 scenarios(simulator => 1);
 
 compile(
-    verilator_flags2 => ["--timing"],
+    verilator_flags2 => ["--binary --timing"],
     );
 
 ok(1);
diff --git a/test_regress/t/t_math_signed5.pl b/test_regress/t/t_math_signed5.pl
index f5debdb98..39ae2a784 100755
--- a/test_regress/t/t_math_signed5.pl
+++ b/test_regress/t/t_math_signed5.pl
@@ -11,7 +11,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 scenarios(simulator => 1);
 
 compile(
-    verilator_flags2 => ['--no-timing'],
+    verilator_flags2 => ['--binary --no-timing'],
     );
 
 execute(
diff --git a/test_regress/t/t_math_signed5_timing.pl b/test_regress/t/t_math_signed5_timing.pl
index 34482e187..695a67c98 100755
--- a/test_regress/t/t_math_signed5_timing.pl
+++ b/test_regress/t/t_math_signed5_timing.pl
@@ -13,7 +13,7 @@ scenarios(simulator => 1);
 top_filename("t/t_math_signed5.v");
 
 compile(
-    verilator_flags2 => ['--timing'],
+    verilator_flags2 => ['--binary --timing'],
     timing_loop => 1,
     );
 
diff --git a/test_regress/t/t_package_ddecl.pl b/test_regress/t/t_package_ddecl.pl
index f5debdb98..39ae2a784 100755
--- a/test_regress/t/t_package_ddecl.pl
+++ b/test_regress/t/t_package_ddecl.pl
@@ -11,7 +11,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 scenarios(simulator => 1);
 
 compile(
-    verilator_flags2 => ['--no-timing'],
+    verilator_flags2 => ['--binary --no-timing'],
     );
 
 execute(
diff --git a/test_regress/t/t_package_ddecl_timing.pl b/test_regress/t/t_package_ddecl_timing.pl
index f7858dcb8..85bccb16b 100755
--- a/test_regress/t/t_package_ddecl_timing.pl
+++ b/test_regress/t/t_package_ddecl_timing.pl
@@ -13,7 +13,7 @@ scenarios(simulator => 1);
 top_filename("t/t_package_ddecl.v");
 
 compile(
-    verilator_flags2 => ['--timing'],
+    verilator_flags2 => ['--binary --timing'],
     timing_loop => 1,
     );
 
diff --git a/test_regress/t/t_parse_delay.pl b/test_regress/t/t_parse_delay.pl
index 40a477a35..d399f16df 100755
--- a/test_regress/t/t_parse_delay.pl
+++ b/test_regress/t/t_parse_delay.pl
@@ -11,7 +11,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 scenarios(simulator => 1);
 
 compile(
-    verilator_flags2 => ['--no-timing'],
+    verilator_flags2 => ['--binary --no-timing'],
     );
 
 ok(1);
diff --git a/test_regress/t/t_parse_delay_timing.pl b/test_regress/t/t_parse_delay_timing.pl
index d466d38c7..c5543a880 100755
--- a/test_regress/t/t_parse_delay_timing.pl
+++ b/test_regress/t/t_parse_delay_timing.pl
@@ -13,7 +13,7 @@ scenarios(simulator => 1);
 top_filename("t/t_parse_delay.v");
 
 compile(
-    verilator_flags2 => ['--timing'],
+    verilator_flags2 => ['--binary --timing'],
     );
 
 ok(1);
diff --git a/test_regress/t/t_process.pl b/test_regress/t/t_process.pl
index 7eac3054e..1ef7ad890 100755
--- a/test_regress/t/t_process.pl
+++ b/test_regress/t/t_process.pl
@@ -11,7 +11,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 scenarios(simulator => 1);
 
 compile(
-    v_flags2 => ["--timing"],
+    v_flags2 => ["--binary --timing"],
     );
 
 execute(
diff --git a/test_regress/t/t_process_fork.pl b/test_regress/t/t_process_fork.pl
index 604f632d4..67c7c9e67 100755
--- a/test_regress/t/t_process_fork.pl
+++ b/test_regress/t/t_process_fork.pl
@@ -11,7 +11,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 scenarios(simulator => 1);
 
 compile(
-    verilator_flags2 => ["--timing"],
+    verilator_flags2 => ["--binary --timing"],
     );
 
 execute(
diff --git a/test_regress/t/t_process_rand.pl b/test_regress/t/t_process_rand.pl
index 4152e3d9b..eb241c303 100755
--- a/test_regress/t/t_process_rand.pl
+++ b/test_regress/t/t_process_rand.pl
@@ -11,7 +11,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 scenarios(simulator => 1);
 
 compile(
-    v_flags2 => ["--timing"],
+    v_flags2 => ["--binary --timing"],
     );
 
 execute(
diff --git a/test_regress/t/t_process_std.pl b/test_regress/t/t_process_std.pl
index c9318acf2..207d1490a 100755
--- a/test_regress/t/t_process_std.pl
+++ b/test_regress/t/t_process_std.pl
@@ -13,7 +13,7 @@ scenarios(simulator => 1);
 top_filename("t/t_process.v");
 
 compile(
-    v_flags2 => ["+define+T_PROCESS+std::process", "--timing"],
+    v_flags2 => ["--binary --timing", "+define+T_PROCESS+std::process"],
     );
 
 execute(
diff --git a/test_regress/t/t_queue_persistence_inl_unsup.pl b/test_regress/t/t_queue_persistence_inl_unsup.pl
index 8cf7fc5b6..bbbc342e1 100755
--- a/test_regress/t/t_queue_persistence_inl_unsup.pl
+++ b/test_regress/t/t_queue_persistence_inl_unsup.pl
@@ -18,7 +18,7 @@ if (!$Self->have_coroutines) {
 else {
     compile(
         timing_loop => 1,
-        verilator_flags2 => ["--timing"],
+        verilator_flags2 => ["--binary --timing"],
         fails => 1,   # bug3385 need to fix "ref"
         expect_filename => $Self->{golden_filename},
         );
diff --git a/test_regress/t/t_queue_persistence_noinl.pl b/test_regress/t/t_queue_persistence_noinl.pl
index 187562462..2f2670fa9 100755
--- a/test_regress/t/t_queue_persistence_noinl.pl
+++ b/test_regress/t/t_queue_persistence_noinl.pl
@@ -18,7 +18,7 @@ if (!$Self->have_coroutines) {
 else {
     compile(
         timing_loop => 1,
-        verilator_flags2 => ["--timing --fno-inline +define+TEST_NOINLINE"],
+        verilator_flags2 => ["--binary --timing --fno-inline +define+TEST_NOINLINE"],
         );
 
     execute(
diff --git a/test_regress/t/t_selextract_in_paramextends.pl b/test_regress/t/t_selextract_in_paramextends.pl
index b267631e9..fec4c2942 100755
--- a/test_regress/t/t_selextract_in_paramextends.pl
+++ b/test_regress/t/t_selextract_in_paramextends.pl
@@ -11,7 +11,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 scenarios(simulator => 1);
 
 compile(
-    verilator_flags2 => ["--timing"],
+    verilator_flags2 => ["--binary --timing"],
     );
 
 execute(
diff --git a/test_regress/t/t_semaphore_class.pl b/test_regress/t/t_semaphore_class.pl
index 6e30bd25d..91589388d 100755
--- a/test_regress/t/t_semaphore_class.pl
+++ b/test_regress/t/t_semaphore_class.pl
@@ -11,7 +11,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 scenarios(simulator => 1);
 
 compile(
-    verilator_flags2 => ["--timing"],
+    verilator_flags2 => ["--binary --timing"],
     );
 
 ok(1);
diff --git a/test_regress/t/t_suspendable_deep.pl b/test_regress/t/t_suspendable_deep.pl
index e7824bce9..3b4f977f4 100755
--- a/test_regress/t/t_suspendable_deep.pl
+++ b/test_regress/t/t_suspendable_deep.pl
@@ -11,7 +11,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 scenarios(vlt => 1);
 
 compile(
-    verilator_flags2 => ["--timing"],
+    verilator_flags2 => ["--binary --timing"],
     );
 
 ok(1);
diff --git a/test_regress/t/t_uvm_all.pl b/test_regress/t/t_uvm_all.pl
index 2b328c903..77aac6e6b 100755
--- a/test_regress/t/t_uvm_all.pl
+++ b/test_regress/t/t_uvm_all.pl
@@ -11,7 +11,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 scenarios(vlt => 1);
 
 compile(
-    v_flags2 => ["--timing",
+    v_flags2 => ["--binary --timing",
                  "-Wno-PKGNODECL -Wno-IMPLICITSTATIC -Wno-CONSTRAINTIGN -Wno-MISINDENT",
                  "-Wno-WIDTHEXPAND -Wno-WIDTHTRUNC -Wno-CASTCONST -Wno-REALCVT",
                  "--error-limit 200 --debug-exit-uvm"],
diff --git a/test_regress/t/t_var_in_fork.pl b/test_regress/t/t_var_in_fork.pl
index b267631e9..fec4c2942 100755
--- a/test_regress/t/t_var_in_fork.pl
+++ b/test_regress/t/t_var_in_fork.pl
@@ -11,7 +11,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 scenarios(simulator => 1);
 
 compile(
-    verilator_flags2 => ["--timing"],
+    verilator_flags2 => ["--binary --timing"],
     );
 
 execute(
diff --git a/test_regress/t/t_wait_order.pl b/test_regress/t/t_wait_order.pl
index 2e33e5676..eba12604c 100755
--- a/test_regress/t/t_wait_order.pl
+++ b/test_regress/t/t_wait_order.pl
@@ -12,7 +12,7 @@ scenarios(simulator => 1);
 
 compile(
     expect_filename => $Self->{golden_filename},
-    verilator_flags2 => ['--timing'],
+    verilator_flags2 => ['--binary --timing'],
     fails => $Self->{vlt_all},
     );
 
diff --git a/test_regress/t/t_wait_timing.pl b/test_regress/t/t_wait_timing.pl
index 41fd8a2dd..8791bb4a2 100755
--- a/test_regress/t/t_wait_timing.pl
+++ b/test_regress/t/t_wait_timing.pl
@@ -14,7 +14,7 @@ top_filename("t/t_wait.v");
 
 compile(
     timing_loop => 1,
-    verilator_flags2 => ["--timing -Wno-WAITCONST"],
+    verilator_flags2 => ["--binary --timing -Wno-WAITCONST"],
     );
 
 execute(
diff --git a/test_regress/t/t_while_timing_control.pl b/test_regress/t/t_while_timing_control.pl
index b6091c571..c6aaa67e5 100755
--- a/test_regress/t/t_while_timing_control.pl
+++ b/test_regress/t/t_while_timing_control.pl
@@ -14,7 +14,7 @@ $Self->{main_time_multiplier} = 10e-7 / 10e-9;
 
 compile(
     timing_loop => 1,
-    verilator_flags2 => ['--timing -Wno-ZERODLY'],
+    verilator_flags2 => ['--binary --timing -Wno-ZERODLY'],
     );
 
 execute(