[v2,28/37] GDB: testsuite: reverse: Don't return -1 from top-level (sed)

Message ID 20260429230432.60487-29-thiago.bauermann@linaro.org
State New
Headers
Series GDB: testsuite: Fix top-level returns |

Commit Message

Thiago Jung Bauermann April 29, 2026, 11:04 p.m. UTC
  This patch is purely the result of running:

$ sed -i 's/^    return -1/    return/' *.exp*

inside gdb/testsuite/gdb.reverse.
---
 gdb/testsuite/gdb.reverse/aarch64-mops.exp              | 4 ++--
 gdb/testsuite/gdb.reverse/amd64-tailcall-reverse.exp    | 4 ++--
 gdb/testsuite/gdb.reverse/basic-record-full.exp         | 2 +-
 gdb/testsuite/gdb.reverse/break-precsave.exp            | 2 +-
 gdb/testsuite/gdb.reverse/break-reverse.exp             | 2 +-
 gdb/testsuite/gdb.reverse/consecutive-precsave.exp      | 2 +-
 gdb/testsuite/gdb.reverse/consecutive-reverse.exp       | 2 +-
 gdb/testsuite/gdb.reverse/finish-precsave.exp           | 2 +-
 gdb/testsuite/gdb.reverse/finish-reverse-bkpt.exp       | 2 +-
 gdb/testsuite/gdb.reverse/finish-reverse-next.exp       | 2 +-
 gdb/testsuite/gdb.reverse/finish-reverse.exp            | 2 +-
 gdb/testsuite/gdb.reverse/fstatat-reverse.exp           | 2 +-
 gdb/testsuite/gdb.reverse/getrandom.exp                 | 2 +-
 gdb/testsuite/gdb.reverse/getresuid-reverse.exp         | 2 +-
 gdb/testsuite/gdb.reverse/i386-avx-reverse.exp          | 2 +-
 gdb/testsuite/gdb.reverse/i386-precsave.exp             | 2 +-
 gdb/testsuite/gdb.reverse/i386-reverse.exp              | 2 +-
 gdb/testsuite/gdb.reverse/i386-sse-reverse.exp          | 2 +-
 gdb/testsuite/gdb.reverse/i387-env-reverse.exp          | 2 +-
 gdb/testsuite/gdb.reverse/i387-stack-reverse.exp        | 2 +-
 gdb/testsuite/gdb.reverse/insn-reverse.exp              | 2 +-
 gdb/testsuite/gdb.reverse/machinestate-precsave.exp     | 2 +-
 gdb/testsuite/gdb.reverse/machinestate.exp              | 2 +-
 gdb/testsuite/gdb.reverse/maint-print-instruction.exp   | 2 +-
 gdb/testsuite/gdb.reverse/map-to-same-line.exp          | 4 ++--
 gdb/testsuite/gdb.reverse/next-reverse-bkpt-over-sr.exp | 2 +-
 gdb/testsuite/gdb.reverse/pipe-reverse.exp              | 2 +-
 gdb/testsuite/gdb.reverse/ppc_record_test_isa_2_06.exp  | 2 +-
 gdb/testsuite/gdb.reverse/ppc_record_test_isa_3_1.exp   | 2 +-
 gdb/testsuite/gdb.reverse/readv-reverse.exp             | 2 +-
 gdb/testsuite/gdb.reverse/recursion.exp                 | 2 +-
 gdb/testsuite/gdb.reverse/recvmsg-reverse.exp           | 2 +-
 gdb/testsuite/gdb.reverse/rerun-prec.exp                | 2 +-
 gdb/testsuite/gdb.reverse/s390-mvcle.exp                | 2 +-
 gdb/testsuite/gdb.reverse/sigall-precsave.exp           | 2 +-
 gdb/testsuite/gdb.reverse/sigall-reverse.exp            | 2 +-
 gdb/testsuite/gdb.reverse/singlejmp-reverse.exp         | 2 +-
 gdb/testsuite/gdb.reverse/solib-precsave.exp            | 6 +++---
 gdb/testsuite/gdb.reverse/solib-reverse.exp             | 6 +++---
 gdb/testsuite/gdb.reverse/step-indirect-call-thunk.exp  | 4 ++--
 gdb/testsuite/gdb.reverse/step-precsave.exp             | 2 +-
 gdb/testsuite/gdb.reverse/step-reverse.exp              | 2 +-
 gdb/testsuite/gdb.reverse/test_ioctl_TCSETSW.exp        | 2 +-
 gdb/testsuite/gdb.reverse/until-precsave.exp            | 2 +-
 gdb/testsuite/gdb.reverse/until-reverse.exp             | 2 +-
 gdb/testsuite/gdb.reverse/waitpid-reverse.exp           | 2 +-
 gdb/testsuite/gdb.reverse/watch-precsave.exp            | 2 +-
 gdb/testsuite/gdb.reverse/watch-reverse.exp             | 2 +-
 48 files changed, 56 insertions(+), 56 deletions(-)
  

Patch

diff --git a/gdb/testsuite/gdb.reverse/aarch64-mops.exp b/gdb/testsuite/gdb.reverse/aarch64-mops.exp
index 14602d4feb8a..97f1efe82994 100644
--- a/gdb/testsuite/gdb.reverse/aarch64-mops.exp
+++ b/gdb/testsuite/gdb.reverse/aarch64-mops.exp
@@ -35,15 +35,15 @@  require allow_aarch64_mops_tests
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile} \
 	  [list debug additional_flags=-march=armv9.3-a]] } {
-    return -1
+    return
 }
 
 if {![runto_main]} {
-    return -1
+    return
 }
 
 gdb_test_no_output "record full"
 
 foreach_with_prefix insn_prefix {"set" "cpy" "cpyf"} {
diff --git a/gdb/testsuite/gdb.reverse/amd64-tailcall-reverse.exp b/gdb/testsuite/gdb.reverse/amd64-tailcall-reverse.exp
index 6db4b43ac0a1..c3289fa4c6ff 100644
--- a/gdb/testsuite/gdb.reverse/amd64-tailcall-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/amd64-tailcall-reverse.exp
@@ -25,15 +25,15 @@  if {[info exists COMPILE]} {
 } else {
     require is_x86_64_m64_target
 }
 
 if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile} $opts] } {
-    return -1
+    return
 }
 
 if {![runto_main]} {
-    return -1
+    return
 }
 
 if {[supports_process_record]} {
     gdb_test_no_output "record"
 }
diff --git a/gdb/testsuite/gdb.reverse/basic-record-full.exp b/gdb/testsuite/gdb.reverse/basic-record-full.exp
index 78d6f4d02af6..67655417db28 100644
--- a/gdb/testsuite/gdb.reverse/basic-record-full.exp
+++ b/gdb/testsuite/gdb.reverse/basic-record-full.exp
@@ -22,11 +22,11 @@ 
 require supports_process_record
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 gdb_test_no_output "record" "turn on process record"
diff --git a/gdb/testsuite/gdb.reverse/break-precsave.exp b/gdb/testsuite/gdb.reverse/break-precsave.exp
index bee9fd548f60..994457d4439c 100644
--- a/gdb/testsuite/gdb.reverse/break-precsave.exp
+++ b/gdb/testsuite/gdb.reverse/break-precsave.exp
@@ -21,11 +21,11 @@  require supports_process_record
 
 standard_testfile break-reverse.c
 set precsave [standard_output_file break.precsave]
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 set foo_location  [gdb_get_line_number "break in foo" ]
 set bar_location  [gdb_get_line_number "break in bar" ]
 set main_location [gdb_get_line_number "break in main"]
diff --git a/gdb/testsuite/gdb.reverse/break-reverse.exp b/gdb/testsuite/gdb.reverse/break-reverse.exp
index 5aeaf22e6ada..7ba681828d33 100644
--- a/gdb/testsuite/gdb.reverse/break-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/break-reverse.exp
@@ -19,11 +19,11 @@ 
 require supports_reverse
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 set foo_location  [gdb_get_line_number "break in foo" ]
 set bar_location  [gdb_get_line_number "break in bar" ]
 set main_location [gdb_get_line_number "break in main"]
diff --git a/gdb/testsuite/gdb.reverse/consecutive-precsave.exp b/gdb/testsuite/gdb.reverse/consecutive-precsave.exp
index 7d57661055e9..9a9132d9a4f1 100644
--- a/gdb/testsuite/gdb.reverse/consecutive-precsave.exp
+++ b/gdb/testsuite/gdb.reverse/consecutive-precsave.exp
@@ -21,11 +21,11 @@  require supports_process_record
 
 standard_testfile consecutive-reverse.c
 set precsave [standard_output_file consecutive.precsave]
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 # Activate process record/replay
diff --git a/gdb/testsuite/gdb.reverse/consecutive-reverse.exp b/gdb/testsuite/gdb.reverse/consecutive-reverse.exp
index c22cb96504b4..1b44a134cb84 100644
--- a/gdb/testsuite/gdb.reverse/consecutive-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/consecutive-reverse.exp
@@ -19,11 +19,11 @@ 
 require supports_reverse
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 if {[supports_process_record]} {
diff --git a/gdb/testsuite/gdb.reverse/finish-precsave.exp b/gdb/testsuite/gdb.reverse/finish-precsave.exp
index 7430b890d545..00c7d8caa419 100644
--- a/gdb/testsuite/gdb.reverse/finish-precsave.exp
+++ b/gdb/testsuite/gdb.reverse/finish-precsave.exp
@@ -20,11 +20,11 @@  require supports_reverse
 
 standard_testfile finish-reverse.c
 set precsave [standard_output_file finish.precsave]
 
 if { [prepare_for_testing "failed to prepare" "$testfile" $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 if {[supports_process_record]} {
diff --git a/gdb/testsuite/gdb.reverse/finish-reverse-bkpt.exp b/gdb/testsuite/gdb.reverse/finish-reverse-bkpt.exp
index 67df198bb051..19ebebf6a126 100644
--- a/gdb/testsuite/gdb.reverse/finish-reverse-bkpt.exp
+++ b/gdb/testsuite/gdb.reverse/finish-reverse-bkpt.exp
@@ -43,11 +43,11 @@ 
 require supports_reverse
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" "$testfile" $srcfile] } {
-    return -1
+    return
 }
 
 if {![runto_main]} {
     return 0
 }
diff --git a/gdb/testsuite/gdb.reverse/finish-reverse-next.exp b/gdb/testsuite/gdb.reverse/finish-reverse-next.exp
index 325282855c64..96af034af1ef 100644
--- a/gdb/testsuite/gdb.reverse/finish-reverse-next.exp
+++ b/gdb/testsuite/gdb.reverse/finish-reverse-next.exp
@@ -46,11 +46,11 @@  if {![supports_reverse]} {
 }
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 if {[supports_process_record]} {
diff --git a/gdb/testsuite/gdb.reverse/finish-reverse.exp b/gdb/testsuite/gdb.reverse/finish-reverse.exp
index 3f3ef666323a..b234d7fec143 100644
--- a/gdb/testsuite/gdb.reverse/finish-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/finish-reverse.exp
@@ -19,11 +19,11 @@ 
 require supports_reverse
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" "$testfile" $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 if {[supports_process_record]} {
diff --git a/gdb/testsuite/gdb.reverse/fstatat-reverse.exp b/gdb/testsuite/gdb.reverse/fstatat-reverse.exp
index 7ccdcd4ae9e8..4c8dfed07b17 100644
--- a/gdb/testsuite/gdb.reverse/fstatat-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/fstatat-reverse.exp
@@ -22,11 +22,11 @@ 
 require supports_reverse
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 if {[supports_process_record]} {
diff --git a/gdb/testsuite/gdb.reverse/getrandom.exp b/gdb/testsuite/gdb.reverse/getrandom.exp
index 410d4ab3bde6..c65de709c4ad 100644
--- a/gdb/testsuite/gdb.reverse/getrandom.exp
+++ b/gdb/testsuite/gdb.reverse/getrandom.exp
@@ -23,11 +23,11 @@  require supports_reverse
 require {have_system_header sys/random.h}
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 if {[supports_process_record]} {
diff --git a/gdb/testsuite/gdb.reverse/getresuid-reverse.exp b/gdb/testsuite/gdb.reverse/getresuid-reverse.exp
index af8311344bfb..27d10afc4e91 100644
--- a/gdb/testsuite/gdb.reverse/getresuid-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/getresuid-reverse.exp
@@ -22,11 +22,11 @@ 
 require supports_reverse
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 if {[supports_process_record]} {
diff --git a/gdb/testsuite/gdb.reverse/i386-avx-reverse.exp b/gdb/testsuite/gdb.reverse/i386-avx-reverse.exp
index bdb76733e959..2268f2c28722 100644
--- a/gdb/testsuite/gdb.reverse/i386-avx-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/i386-avx-reverse.exp
@@ -39,11 +39,11 @@  standard_testfile
 set additional_flags [gdb_target_symbol_prefix_flags]
 lappend_include_file alloc_lib $srcdir/lib/precise-aligned-alloc.c
 
 if {[prepare_for_testing "failed to prepare" $testfile $srcfile \
 	 [list debug $additional_flags $alloc_lib]]} {
-    return -1
+    return
 }
 
 # Shorthand to test reversing through one instruction and
 # testing if a register has the expected value.
 # Prefix, if included, should end with a colon and space.
diff --git a/gdb/testsuite/gdb.reverse/i386-precsave.exp b/gdb/testsuite/gdb.reverse/i386-precsave.exp
index 4841c1b65aa2..4bde399bc457 100644
--- a/gdb/testsuite/gdb.reverse/i386-precsave.exp
+++ b/gdb/testsuite/gdb.reverse/i386-precsave.exp
@@ -28,11 +28,11 @@  set precsave [standard_output_file i386.precsave]
 # some targets have leading underscores on assembly symbols.
 set additional_flags [gdb_target_symbol_prefix_flags]
 
 if {[prepare_for_testing "failed to prepare" $testfile $srcfile \
 	 [list debug $additional_flags]]} {
-    return -1
+    return
 }
 
 set end_of_main          [gdb_get_line_number " end of main "]
 set end_of_inc_dec_tests [gdb_get_line_number " end inc_dec_tests "]
 
diff --git a/gdb/testsuite/gdb.reverse/i386-reverse.exp b/gdb/testsuite/gdb.reverse/i386-reverse.exp
index 0d27454d0101..2013a969f8ca 100644
--- a/gdb/testsuite/gdb.reverse/i386-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/i386-reverse.exp
@@ -26,11 +26,11 @@  standard_testfile
 # some targets have leading underscores on assembly symbols.
 set additional_flags [gdb_target_symbol_prefix_flags]
 
 if {[prepare_for_testing "failed to prepare" $testfile $srcfile \
 	 [list debug $additional_flags]]} {
-    return -1
+    return
 }
 
 set end_of_main          [gdb_get_line_number " end of main "]
 set end_of_inc_dec_tests [gdb_get_line_number " end inc_dec_tests "]
 
diff --git a/gdb/testsuite/gdb.reverse/i386-sse-reverse.exp b/gdb/testsuite/gdb.reverse/i386-sse-reverse.exp
index ad8eaeaadc23..dc1308861e2c 100644
--- a/gdb/testsuite/gdb.reverse/i386-sse-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/i386-sse-reverse.exp
@@ -31,11 +31,11 @@  standard_testfile
 # some targets have leading underscores on assembly symbols.
 set additional_flags [gdb_target_symbol_prefix_flags]
 
 if {[prepare_for_testing "failed to prepare" $testfile $srcfile \
 	 [list debug $additional_flags]]} {
-    return -1
+    return
 }
 
 set end_of_main          [gdb_get_line_number " end of main "]
 set end_sse_test         [gdb_get_line_number " end sse_test "]
 set end_ssse3_test       [gdb_get_line_number " end ssse3_test "]
diff --git a/gdb/testsuite/gdb.reverse/i387-env-reverse.exp b/gdb/testsuite/gdb.reverse/i387-env-reverse.exp
index 5fc6346f86bf..ebf7d8dec425 100644
--- a/gdb/testsuite/gdb.reverse/i387-env-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/i387-env-reverse.exp
@@ -23,11 +23,11 @@  standard_testfile
 # some targets have leading underscores on assembly symbols.
 set additional_flags [gdb_target_symbol_prefix_flags]
 
 if {[prepare_for_testing "failed to prepare" $testfile $srcfile \
 	 [list debug $additional_flags]]} {
-    return -1
+    return
 }
 
 runto_main
 
 # Record to end of main
diff --git a/gdb/testsuite/gdb.reverse/i387-stack-reverse.exp b/gdb/testsuite/gdb.reverse/i387-stack-reverse.exp
index c6d3cca23424..384faade5eee 100644
--- a/gdb/testsuite/gdb.reverse/i387-stack-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/i387-stack-reverse.exp
@@ -23,11 +23,11 @@  standard_testfile
 # some targets have leading underscores on assembly symbols.
 set additional_flags [gdb_target_symbol_prefix_flags]
 
 if {[prepare_for_testing "failed to prepare" $testfile $srcfile \
 	 [list debug $additional_flags]]} {
-    return -1
+    return
 }
 
 runto_main
 
 # Record to end of main
diff --git a/gdb/testsuite/gdb.reverse/insn-reverse.exp b/gdb/testsuite/gdb.reverse/insn-reverse.exp
index c92059143ece..5b4f6a28f7af 100644
--- a/gdb/testsuite/gdb.reverse/insn-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/insn-reverse.exp
@@ -17,11 +17,11 @@  require supports_reverse !use_gdb_stub
 
 standard_testfile
 
 if {[prepare_for_testing "failed to prepare" $testfile $srcfile \
 	 [list debug]]} {
-    return -1
+    return
 }
 if {![runto_main]} {
     return
 }
 
diff --git a/gdb/testsuite/gdb.reverse/machinestate-precsave.exp b/gdb/testsuite/gdb.reverse/machinestate-precsave.exp
index 74c041143d87..8a11c6d24b98 100644
--- a/gdb/testsuite/gdb.reverse/machinestate-precsave.exp
+++ b/gdb/testsuite/gdb.reverse/machinestate-precsave.exp
@@ -41,11 +41,11 @@  require supports_process_record
 standard_testfile machinestate.c ms1.c
 set precsave [standard_output_file machinestate.precsave]
 
 if { [prepare_for_testing "failed to prepare" $testfile \
 	  [list $srcfile $srcfile2]] } {
-    return -1
+    return
 }
 
 set newline "\[\r\n\]+"
 
 set beginmain [gdb_get_line_number " begin main " $srcfile]
diff --git a/gdb/testsuite/gdb.reverse/machinestate.exp b/gdb/testsuite/gdb.reverse/machinestate.exp
index 0026af7704bd..de16fbcbb23a 100644
--- a/gdb/testsuite/gdb.reverse/machinestate.exp
+++ b/gdb/testsuite/gdb.reverse/machinestate.exp
@@ -39,11 +39,11 @@  require supports_reverse
 
 standard_testfile .c ms1.c
 
 if { [prepare_for_testing "failed to prepare" $testfile \
 	  [list $srcfile $srcfile2]] } {
-    return -1
+    return
 }
 
 set newline "\[\r\n\]+"
 
 set beginmain [gdb_get_line_number " begin main " $srcfile]
diff --git a/gdb/testsuite/gdb.reverse/maint-print-instruction.exp b/gdb/testsuite/gdb.reverse/maint-print-instruction.exp
index dc3bb6b547af..ad8009b415a9 100644
--- a/gdb/testsuite/gdb.reverse/maint-print-instruction.exp
+++ b/gdb/testsuite/gdb.reverse/maint-print-instruction.exp
@@ -28,11 +28,11 @@  if {![supports_reverse]} {
 }
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 proc test_print { has_history level test_name } {
     gdb_test_multiple "maint print record-instruction $level" $test_name {
 	-re -wrap ".*Not enough recorded history.*" {
diff --git a/gdb/testsuite/gdb.reverse/map-to-same-line.exp b/gdb/testsuite/gdb.reverse/map-to-same-line.exp
index 4e3c1a50229b..1cfb060dfaab 100644
--- a/gdb/testsuite/gdb.reverse/map-to-same-line.exp
+++ b/gdb/testsuite/gdb.reverse/map-to-same-line.exp
@@ -31,11 +31,11 @@  require is_c_compiler_gcc
 require supports_reverse
 
 standard_testfile .c .S
 
 if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile}] } {
-    return -1
+    return
 }
 
 set asm_file [standard_output_file $srcfile2]
 Dwarf::assemble $asm_file {
     global srcdir subdir srcfile
@@ -107,11 +107,11 @@  Dwarf::assemble $asm_file {
     }
 }
 
 if { [prepare_for_testing "failed to prepare" ${testfile} \
 	[list $srcfile $asm_file] {nodebug} ] } {
-    return -1
+    return
 }
 
 if { ![runto_main] } {
     return
 }
diff --git a/gdb/testsuite/gdb.reverse/next-reverse-bkpt-over-sr.exp b/gdb/testsuite/gdb.reverse/next-reverse-bkpt-over-sr.exp
index 83162e59dfc6..d34fe130e8e5 100644
--- a/gdb/testsuite/gdb.reverse/next-reverse-bkpt-over-sr.exp
+++ b/gdb/testsuite/gdb.reverse/next-reverse-bkpt-over-sr.exp
@@ -43,11 +43,11 @@ 
 require supports_reverse
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 if {![runto_main]} {
     return 0
 }
diff --git a/gdb/testsuite/gdb.reverse/pipe-reverse.exp b/gdb/testsuite/gdb.reverse/pipe-reverse.exp
index d4b99c91a8e7..67035866af9a 100644
--- a/gdb/testsuite/gdb.reverse/pipe-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/pipe-reverse.exp
@@ -22,11 +22,11 @@ 
 require supports_reverse
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 if {[supports_process_record]} {
diff --git a/gdb/testsuite/gdb.reverse/ppc_record_test_isa_2_06.exp b/gdb/testsuite/gdb.reverse/ppc_record_test_isa_2_06.exp
index aa69e5d00c99..6bbe98116385 100644
--- a/gdb/testsuite/gdb.reverse/ppc_record_test_isa_2_06.exp
+++ b/gdb/testsuite/gdb.reverse/ppc_record_test_isa_2_06.exp
@@ -38,11 +38,11 @@  set executable record_test_isa_2_06
 set options [list debug]
 
 require {istarget "powerpc*"}
 
 if {[build_executable "failed to prepare" $executable $srcfile $options] == -1} {
-    return -1
+    return
 }
 
 clean_restart $executable
 
 if {![runto_main]} {
diff --git a/gdb/testsuite/gdb.reverse/ppc_record_test_isa_3_1.exp b/gdb/testsuite/gdb.reverse/ppc_record_test_isa_3_1.exp
index d4ecbbb7fb66..8435d3d840d6 100644
--- a/gdb/testsuite/gdb.reverse/ppc_record_test_isa_3_1.exp
+++ b/gdb/testsuite/gdb.reverse/ppc_record_test_isa_3_1.exp
@@ -39,11 +39,11 @@  set executable record_test_isa_3_1
 
 require {istarget "powerpc*"} allow_power_isa_3_1_tests
 
 set options [list additional_flags=-mcpu=power10 debug]
 if {[build_executable "failed to prepare" $executable $srcfile $options] == -1} {
-    return -1
+    return
 }
 
 clean_restart $executable
 
 if {![runto_main]} {
diff --git a/gdb/testsuite/gdb.reverse/readv-reverse.exp b/gdb/testsuite/gdb.reverse/readv-reverse.exp
index e05e3de351b6..e5467d8051f8 100644
--- a/gdb/testsuite/gdb.reverse/readv-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/readv-reverse.exp
@@ -22,11 +22,11 @@ 
 require supports_reverse
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 if {[supports_process_record]} {
diff --git a/gdb/testsuite/gdb.reverse/recursion.exp b/gdb/testsuite/gdb.reverse/recursion.exp
index 6284ebc0c16e..6a81821d62bd 100644
--- a/gdb/testsuite/gdb.reverse/recursion.exp
+++ b/gdb/testsuite/gdb.reverse/recursion.exp
@@ -19,11 +19,11 @@ 
 require supports_reverse
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 if {[supports_process_record]} {
diff --git a/gdb/testsuite/gdb.reverse/recvmsg-reverse.exp b/gdb/testsuite/gdb.reverse/recvmsg-reverse.exp
index ce91b59e7edc..3fa433dde569 100644
--- a/gdb/testsuite/gdb.reverse/recvmsg-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/recvmsg-reverse.exp
@@ -22,11 +22,11 @@ 
 require supports_reverse
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 if {[supports_process_record]} {
diff --git a/gdb/testsuite/gdb.reverse/rerun-prec.exp b/gdb/testsuite/gdb.reverse/rerun-prec.exp
index aed3c9e52266..a1a41b007561 100644
--- a/gdb/testsuite/gdb.reverse/rerun-prec.exp
+++ b/gdb/testsuite/gdb.reverse/rerun-prec.exp
@@ -16,11 +16,11 @@ 
 require supports_reverse supports_process_record
 
 standard_testfile
 
 if {[prepare_for_testing "failed to prepare" $testfile [list $srcfile]]} {
-    return -1
+    return
 }
 
 # The bug is a regression in the sequence "run; record; run".
 runto_main
 gdb_test_no_output "record" "turn on process record"
diff --git a/gdb/testsuite/gdb.reverse/s390-mvcle.exp b/gdb/testsuite/gdb.reverse/s390-mvcle.exp
index 4bb89c678b48..12fd622f08db 100644
--- a/gdb/testsuite/gdb.reverse/s390-mvcle.exp
+++ b/gdb/testsuite/gdb.reverse/s390-mvcle.exp
@@ -20,11 +20,11 @@ 
 require supports_reverse {istarget "s390*-*-*"}
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 if {[supports_process_record]} {
diff --git a/gdb/testsuite/gdb.reverse/sigall-precsave.exp b/gdb/testsuite/gdb.reverse/sigall-precsave.exp
index 2635e6fdcda4..d8bc6dfaccfa 100644
--- a/gdb/testsuite/gdb.reverse/sigall-precsave.exp
+++ b/gdb/testsuite/gdb.reverse/sigall-precsave.exp
@@ -19,11 +19,11 @@  require supports_reverse
 
 standard_testfile sigall-reverse.c
 set precsave [standard_output_file sigall.precsave]
 
 if {[build_executable $testfile.exp $testfile $srcfile debug] == -1} {
-    return -1
+    return
 }
 
 proc test_one_sig {nextsig} {
     global sig_supported
     global gdb_prompt
diff --git a/gdb/testsuite/gdb.reverse/sigall-reverse.exp b/gdb/testsuite/gdb.reverse/sigall-reverse.exp
index 5f0ab640d186..2dae1933d818 100644
--- a/gdb/testsuite/gdb.reverse/sigall-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/sigall-reverse.exp
@@ -18,11 +18,11 @@  require {!target_info exists gdb,nosignals}
 require supports_reverse
 
 standard_testfile
 
 if {[build_executable $testfile.exp $testfile $srcfile debug] == -1} {
-    return -1
+    return
 }
 
 proc test_one_sig {nextsig} {
     global sig_supported
     global gdb_prompt
diff --git a/gdb/testsuite/gdb.reverse/singlejmp-reverse.exp b/gdb/testsuite/gdb.reverse/singlejmp-reverse.exp
index 00fd8b8bd957..830292cd6669 100644
--- a/gdb/testsuite/gdb.reverse/singlejmp-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/singlejmp-reverse.exp
@@ -35,11 +35,11 @@  if {[info exists COMPILE]} {
 }
 
 clean_restart $executable
 
 if {![runto_main]} {
-    return -1
+    return
 }
 
 if {[supports_process_record]} {
     gdb_test_no_output "record"
 }
diff --git a/gdb/testsuite/gdb.reverse/solib-precsave.exp b/gdb/testsuite/gdb.reverse/solib-precsave.exp
index 0e0dd3668ac7..ef3229c74927 100644
--- a/gdb/testsuite/gdb.reverse/solib-precsave.exp
+++ b/gdb/testsuite/gdb.reverse/solib-precsave.exp
@@ -30,16 +30,16 @@  set library2 [standard_output_file ${lib2file}.sl]
 
 # Compile the first without debug info so that
 # stepping and reverse stepping doesn't end up inside them.
 if { [gdb_compile_shlib ${srcdir}/${subdir}/${lib1src} ${library1} ""] != "" } {
     untested "failed to compile shared library 1"
-    return -1
+    return
 }
 
 if { [gdb_compile_shlib ${srcdir}/${subdir}/${lib2src} ${library2} "debug"] != "" } {
     untested "failed to compile shared library 2"
-    return -1
+    return
 }
 
 set exec_opts [list debug shlib=${library1} shlib=${library2}]
 
 # Attempt to prevent -Wl,-z,relro which may happen by default with some
@@ -48,11 +48,11 @@  set exec_opts [list debug shlib=${library1} shlib=${library2}]
 
 if { [gdb_compile ${srcdir}/${subdir}/${srcfile} ${binfile} executable \
 		     [concat $exec_opts ldflags=-Wl,-z,norelro]] != ""
      && [gdb_compile ${srcdir}/${subdir}/${srcfile} ${binfile} executable $exec_opts] != "" } {
     untested "failed to compile"
-    return -1
+    return
 }
 
 # Note: The test previously did "set debug-file-directory" to (try to)
 # ensure the debug info for the dynamic loader and libc weren't found.
 # This doesn't work if the debug info is in the .debug subdirectory.
diff --git a/gdb/testsuite/gdb.reverse/solib-reverse.exp b/gdb/testsuite/gdb.reverse/solib-reverse.exp
index 04bbf56757cb..c5a610fcb391 100644
--- a/gdb/testsuite/gdb.reverse/solib-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/solib-reverse.exp
@@ -28,23 +28,23 @@  set library2 [standard_output_file ${lib2file}.sl]
 
 # Compile the first without debug info so that
 # stepping and reverse stepping doesn't end up inside them.
 if { [gdb_compile_shlib ${srcdir}/${subdir}/${lib1src} ${library1} ""] != "" } {
     untested "failed to compile shared library 1"
-    return -1
+    return
 }
 
 if { [gdb_compile_shlib ${srcdir}/${subdir}/${lib2src} ${library2} "debug"] != "" } {
     untested "failed to compile shared library 2"
-    return -1
+    return
 }
 
 set exec_opts [list debug shlib=${library1} shlib=${library2}]
 
 if { [gdb_compile ${srcdir}/${subdir}/${srcfile} ${binfile} executable $exec_opts] != "" } {
     untested "failed to compile"
-    return -1
+    return
 }
 
 # Note: The test previously did "set debug-file-directory" to (try to)
 # ensure the debug info for the dynamic loader and libc weren't found.
 # This doesn't work if the debug info is in the .debug subdirectory.
diff --git a/gdb/testsuite/gdb.reverse/step-indirect-call-thunk.exp b/gdb/testsuite/gdb.reverse/step-indirect-call-thunk.exp
index 3b00a2fb0895..feb7a840d8a0 100644
--- a/gdb/testsuite/gdb.reverse/step-indirect-call-thunk.exp
+++ b/gdb/testsuite/gdb.reverse/step-indirect-call-thunk.exp
@@ -29,15 +29,15 @@  if { [gdb_can_simple_compile fcf-protection-none {int main () { return 0; }} \
 	  executable $fcf_protection_none] } {
     lappend cflags $fcf_protection_none
 }
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile $cflags] } {
-    return -1
+    return
 }
 
 if { ![runto_main] } {
-    return -1
+    return
 }
 
 gdb_test_no_output "record"
 gdb_test "next" ".*" "record trace"
 
diff --git a/gdb/testsuite/gdb.reverse/step-precsave.exp b/gdb/testsuite/gdb.reverse/step-precsave.exp
index 33fabd7ec196..761713436771 100644
--- a/gdb/testsuite/gdb.reverse/step-precsave.exp
+++ b/gdb/testsuite/gdb.reverse/step-precsave.exp
@@ -25,11 +25,11 @@  require supports_process_record
 
 standard_testfile step-reverse.c
 set precsave [standard_output_file step.precsave]
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 # Activate process record/replay
diff --git a/gdb/testsuite/gdb.reverse/step-reverse.exp b/gdb/testsuite/gdb.reverse/step-reverse.exp
index b14273e8bcf6..d72638c9169b 100644
--- a/gdb/testsuite/gdb.reverse/step-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/step-reverse.exp
@@ -23,11 +23,11 @@ 
 require supports_reverse
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 if {[supports_process_record]} {
diff --git a/gdb/testsuite/gdb.reverse/test_ioctl_TCSETSW.exp b/gdb/testsuite/gdb.reverse/test_ioctl_TCSETSW.exp
index c3fb0487d187..17d4369ee4ee 100644
--- a/gdb/testsuite/gdb.reverse/test_ioctl_TCSETSW.exp
+++ b/gdb/testsuite/gdb.reverse/test_ioctl_TCSETSW.exp
@@ -19,11 +19,11 @@ 
 require supports_process_record
 
 standard_testfile .c
 
 if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile}] } {
-    return -1
+    return
 }
 
 if {![runto_main]} {
     untested "could not run to main"
     continue
diff --git a/gdb/testsuite/gdb.reverse/until-precsave.exp b/gdb/testsuite/gdb.reverse/until-precsave.exp
index a0bed8dfba51..411fc14fed27 100644
--- a/gdb/testsuite/gdb.reverse/until-precsave.exp
+++ b/gdb/testsuite/gdb.reverse/until-precsave.exp
@@ -22,11 +22,11 @@  require supports_process_record
 standard_testfile until-reverse.c ur1.c
 set precsave [standard_output_file until.precsave]
 
 if { [prepare_for_testing "failed to prepare" $testfile \
 	  [list $srcfile $srcfile2]] } {
-    return -1
+    return
 }
 
 set bp_location1  [gdb_get_line_number "set breakpoint 1 here"]
 set bp_location7  [gdb_get_line_number "set breakpoint 7 here"]
 set bp_location8  [gdb_get_line_number "set breakpoint 8 here" "$srcfile2"]
diff --git a/gdb/testsuite/gdb.reverse/until-reverse.exp b/gdb/testsuite/gdb.reverse/until-reverse.exp
index dc8fcbe78d68..74703223c643 100644
--- a/gdb/testsuite/gdb.reverse/until-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/until-reverse.exp
@@ -20,11 +20,11 @@  require supports_reverse
 
 standard_testfile .c ur1.c
 
 if { [prepare_for_testing "failed to prepare" $testfile \
 	  [list $srcfile $srcfile2]] } {
-    return -1
+    return
 }
 
 set bp_location1  [gdb_get_line_number "set breakpoint 1 here"]
 set bp_location7  [gdb_get_line_number "set breakpoint 7 here"]
 set bp_location8  [gdb_get_line_number "set breakpoint 8 here" "$srcfile2"]
diff --git a/gdb/testsuite/gdb.reverse/waitpid-reverse.exp b/gdb/testsuite/gdb.reverse/waitpid-reverse.exp
index bc6b32cdc663..dc213259bf0f 100644
--- a/gdb/testsuite/gdb.reverse/waitpid-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/waitpid-reverse.exp
@@ -24,11 +24,11 @@ 
 require supports_reverse
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 if {[supports_process_record]} {
diff --git a/gdb/testsuite/gdb.reverse/watch-precsave.exp b/gdb/testsuite/gdb.reverse/watch-precsave.exp
index 2cc668fdc6a5..474012f74c73 100644
--- a/gdb/testsuite/gdb.reverse/watch-precsave.exp
+++ b/gdb/testsuite/gdb.reverse/watch-precsave.exp
@@ -22,11 +22,11 @@  require supports_process_record
 
 standard_testfile watch-reverse.c
 set precsave [standard_output_file watch.precsave]
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 # Activate process record/replay
diff --git a/gdb/testsuite/gdb.reverse/watch-reverse.exp b/gdb/testsuite/gdb.reverse/watch-reverse.exp
index e165aa084dc7..52b36c6f70ae 100644
--- a/gdb/testsuite/gdb.reverse/watch-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/watch-reverse.exp
@@ -21,11 +21,11 @@ 
 require supports_reverse
 
 standard_testfile
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 runto_main
 
 if {[supports_process_record]} {