[v2,05/37] GDB: testsuite: powerpc: Don't return value from top-level (sed)

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

Commit Message

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

$ sed -i 's/^    return -1/    return/' \
      aix*.exp* altivec*.exp* e500*.exp* powerpc*.exp* ppc*.exp* vsx*.exp*

and

$ sed -i 's/^    return 0/    return/' \
      aix*.exp* altivec*.exp* e500*.exp* powerpc*.exp* ppc*.exp* vsx*.exp*

inside gdb/testsuite/gdb.arch.

I didn't include changes made to return statements inside procedures.
---
 gdb/testsuite/gdb.arch/aix-sighandle.exp            |  2 +-
 gdb/testsuite/gdb.arch/altivec-abi.exp              |  2 +-
 gdb/testsuite/gdb.arch/altivec-regs.exp             |  4 ++--
 gdb/testsuite/gdb.arch/e500-abi.exp                 |  2 +-
 gdb/testsuite/gdb.arch/e500-prologue.exp            |  2 +-
 gdb/testsuite/gdb.arch/e500-regs.exp                |  4 ++--
 gdb/testsuite/gdb.arch/powerpc-addpcis.exp          |  2 +-
 gdb/testsuite/gdb.arch/powerpc-aix-prologue.exp     |  2 +-
 gdb/testsuite/gdb.arch/powerpc-altivec.exp          |  2 +-
 gdb/testsuite/gdb.arch/powerpc-altivec2.exp         |  2 +-
 gdb/testsuite/gdb.arch/powerpc-altivec3.exp         |  2 +-
 gdb/testsuite/gdb.arch/powerpc-d128-regs.exp        |  2 +-
 gdb/testsuite/gdb.arch/powerpc-fpscr-gcore.exp      | 10 +++++-----
 gdb/testsuite/gdb.arch/powerpc-lnia.exp             |  2 +-
 gdb/testsuite/gdb.arch/powerpc-plxv-nonrel.exp      |  2 +-
 gdb/testsuite/gdb.arch/powerpc-power10.exp          |  2 +-
 gdb/testsuite/gdb.arch/powerpc-power7.exp           |  2 +-
 gdb/testsuite/gdb.arch/powerpc-power8.exp           |  2 +-
 gdb/testsuite/gdb.arch/powerpc-power9.exp           |  2 +-
 gdb/testsuite/gdb.arch/powerpc-prologue-frame.exp   |  2 +-
 gdb/testsuite/gdb.arch/powerpc-prologue.exp         |  4 ++--
 gdb/testsuite/gdb.arch/powerpc-stackless.exp        |  2 +-
 gdb/testsuite/gdb.arch/powerpc-tar.exp              |  2 +-
 gdb/testsuite/gdb.arch/powerpc-trap.exp             |  4 ++--
 gdb/testsuite/gdb.arch/powerpc-vsx-gcore.exp        | 10 +++++-----
 gdb/testsuite/gdb.arch/powerpc-vsx.exp              |  2 +-
 gdb/testsuite/gdb.arch/powerpc-vsx2.exp             |  2 +-
 gdb/testsuite/gdb.arch/powerpc-vsx3.exp             |  2 +-
 gdb/testsuite/gdb.arch/powerpc64-prologue.exp       |  6 +++---
 gdb/testsuite/gdb.arch/ppc-dfp.exp                  |  6 +++---
 gdb/testsuite/gdb.arch/ppc-fp.exp                   |  4 ++--
 gdb/testsuite/gdb.arch/ppc64-atomic-inst.exp        |  2 +-
 gdb/testsuite/gdb.arch/ppc64-break-on-_exit.exp     |  4 ++--
 gdb/testsuite/gdb.arch/ppc64-isa207-atomic-inst.exp |  2 +-
 gdb/testsuite/gdb.arch/ppc64-symtab-cordic.exp      |  4 ++--
 gdb/testsuite/gdb.arch/vsx-regs.exp                 |  4 ++--
 gdb/testsuite/gdb.arch/vsx-vsr-float28.exp          |  4 ++--
 37 files changed, 58 insertions(+), 58 deletions(-)
  

Patch

diff --git a/gdb/testsuite/gdb.arch/aix-sighandle.exp b/gdb/testsuite/gdb.arch/aix-sighandle.exp
index df988b26a44d..4e5bb50c8560 100644
--- a/gdb/testsuite/gdb.arch/aix-sighandle.exp
+++ b/gdb/testsuite/gdb.arch/aix-sighandle.exp
@@ -14,11 +14,11 @@ 
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
 require {istarget "powerpc*-*-aix*"}
 
 if { [prepare_for_testing "failed to prepare" aix-sighandle aix-sighandle.c] } {
-    return -1
+    return
 }
 
 set srcfile aix-sighandle.c
 set binfile aix-sighandle
 
diff --git a/gdb/testsuite/gdb.arch/altivec-abi.exp b/gdb/testsuite/gdb.arch/altivec-abi.exp
index 40b1c3519e28..f7bea3de4ced 100644
--- a/gdb/testsuite/gdb.arch/altivec-abi.exp
+++ b/gdb/testsuite/gdb.arch/altivec-abi.exp
@@ -168,7 +168,7 @@  if {[test_compiler_info gcc*]} {
     }
 } elseif {[test_compiler_info xlc*]} {
     altivec_abi_tests "additional_flags=-qaltivec" "auto"
 } else {
     warning "unknown compiler"
-    return -1
+    return
 }
diff --git a/gdb/testsuite/gdb.arch/altivec-regs.exp b/gdb/testsuite/gdb.arch/altivec-regs.exp
index 70a15a9190a8..4dd8610d8f15 100644
--- a/gdb/testsuite/gdb.arch/altivec-regs.exp
+++ b/gdb/testsuite/gdb.arch/altivec-regs.exp
@@ -30,11 +30,11 @@  if {[test_compiler_info gcc*]} {
     set compile_flags "$compile_flags additional_flags=-maltivec additional_flags=-mabi=altivec"
 } elseif {[test_compiler_info xlc*]} {
     set compile_flags "$compile_flags additional_flags=-qaltivec"
 } else {
     warning "unknown compiler"
-    return -1
+    return
 }
 
 if  { [gdb_compile ${srcdir}/${subdir}/${srcfile} ${binfile} executable $compile_flags] != "" } {
      untested "failed to compile"
      return -1
@@ -47,11 +47,11 @@  gdb_load ${binfile}
 #
 # Run to `main' where we begin our tests.
 #
 
 if {![runto_main]} {
-    return 0
+    return
 }
 
 gdb_breakpoint [gdb_get_line_number "start here"]
 gdb_continue_to_breakpoint "start here"
 
diff --git a/gdb/testsuite/gdb.arch/e500-abi.exp b/gdb/testsuite/gdb.arch/e500-abi.exp
index 1c074f10aa7e..6f549539e2c3 100644
--- a/gdb/testsuite/gdb.arch/e500-abi.exp
+++ b/gdb/testsuite/gdb.arch/e500-abi.exp
@@ -40,11 +40,11 @@  gdb_load ${binfile}
 #
 # Run to `main' where we begin our tests.
 #
 
 if {![runto_main]} {
-    return 0
+    return
 }
 
 gdb_test "b marker" "Breakpoint 2 at.*file.*e500-abi.c, line ${::decimal}." "break marker"
 gdb_test "continue" "Breakpoint 2.*marker.*e500-abi.c.*" "continue to marker"
 gdb_test "finish" "Run till exit from .0.*marker.*at.*e500-abi.c.*main \\(\\) at.*e500-abi.c.*res_vec = vec_func \\(a_vec,.*goes in r3.*" "back to main (1)"
diff --git a/gdb/testsuite/gdb.arch/e500-prologue.exp b/gdb/testsuite/gdb.arch/e500-prologue.exp
index b3d0056fd97c..f6884285e3a3 100644
--- a/gdb/testsuite/gdb.arch/e500-prologue.exp
+++ b/gdb/testsuite/gdb.arch/e500-prologue.exp
@@ -22,11 +22,11 @@  require {istarget "powerpc-*"}
 set testfile "e500-prologue"
 set srcfile ${testfile}.c
 set binfile ${objdir}/${subdir}/${testfile}
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {}] != "" } {
     unsupported "failed to compile"
-    return -1
+    return
 }
 
 clean_restart
 gdb_load $binfile
 
diff --git a/gdb/testsuite/gdb.arch/e500-regs.exp b/gdb/testsuite/gdb.arch/e500-regs.exp
index 2dc1ade4b448..b9d172c058d7 100644
--- a/gdb/testsuite/gdb.arch/e500-regs.exp
+++ b/gdb/testsuite/gdb.arch/e500-regs.exp
@@ -39,11 +39,11 @@  gdb_load ${binfile}
 #
 # Run to `main' where we begin our tests.
 #
 
 if {![runto_main]} {
-    return 0
+    return
 }
 
 # set all the registers integer portions to 1
 for {set i 0} {$i < 32} {incr i 1} {
 	for {set j 0} {$j < 2} {incr j 1} {
@@ -137,11 +137,11 @@  gdb_expect_list "info vector" ".*$gdb_prompt $" {
 # some unusual values.
 
 clean_restart
 gdb_load $binfile
 if {![runto_main]} {
-    return 0
+    return
 }
 
 gdb_test "break vector_fun" \
  "Breakpoint 2 at.*e500-regs.c, line ${::decimal}\\." \
  "set breakpoint at vector_fun"
diff --git a/gdb/testsuite/gdb.arch/powerpc-addpcis.exp b/gdb/testsuite/gdb.arch/powerpc-addpcis.exp
index e699e730021c..a8895446cde3 100644
--- a/gdb/testsuite/gdb.arch/powerpc-addpcis.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-addpcis.exp
@@ -30,11 +30,11 @@  set retval 0
 
 standard_testfile .s
 
 if { [prepare_for_testing "failed to prepare" $testfile "$srcfile" \
       {debug quiet}] } {
-    return -1
+    return
 }
 
 if {![runto_main]} {
       return
 }
diff --git a/gdb/testsuite/gdb.arch/powerpc-aix-prologue.exp b/gdb/testsuite/gdb.arch/powerpc-aix-prologue.exp
index f465f36fe2da..0da2383b1082 100644
--- a/gdb/testsuite/gdb.arch/powerpc-aix-prologue.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-aix-prologue.exp
@@ -23,11 +23,11 @@  require {istarget "powerpc-*-aix*"}
 set testfile "powerpc-aix-prologue"
 set srcfile ${testfile}.c
 set binfile ${objdir}/${subdir}/${testfile}
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {}] != "" } {
     unsupported "failed to compile"
-    return -1
+    return
 }
 
 clean_restart
 gdb_load $binfile
 
diff --git a/gdb/testsuite/gdb.arch/powerpc-altivec.exp b/gdb/testsuite/gdb.arch/powerpc-altivec.exp
index edbb822ff838..cb89ba7e518f 100644
--- a/gdb/testsuite/gdb.arch/powerpc-altivec.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-altivec.exp
@@ -22,11 +22,11 @@  set objfile [standard_output_file ${testfile}.o]
 
 require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
 gdb_load $objfile
 
diff --git a/gdb/testsuite/gdb.arch/powerpc-altivec2.exp b/gdb/testsuite/gdb.arch/powerpc-altivec2.exp
index 1b34cb6a8a33..621f44dc4e42 100644
--- a/gdb/testsuite/gdb.arch/powerpc-altivec2.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-altivec2.exp
@@ -22,11 +22,11 @@  set objfile [standard_output_file ${testfile}.o]
 
 require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
 gdb_load $objfile
 
diff --git a/gdb/testsuite/gdb.arch/powerpc-altivec3.exp b/gdb/testsuite/gdb.arch/powerpc-altivec3.exp
index 602e65217494..6607e24d7a8f 100644
--- a/gdb/testsuite/gdb.arch/powerpc-altivec3.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-altivec3.exp
@@ -22,11 +22,11 @@  set objfile [standard_output_file ${testfile}.o]
 
 require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
 gdb_load $objfile
 
diff --git a/gdb/testsuite/gdb.arch/powerpc-d128-regs.exp b/gdb/testsuite/gdb.arch/powerpc-d128-regs.exp
index 05ae31877e1b..10c77ef29489 100644
--- a/gdb/testsuite/gdb.arch/powerpc-d128-regs.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-d128-regs.exp
@@ -34,11 +34,11 @@  clean_restart $::testfile
 if {![runto_main]} {
    return
 }
 
 if {[gdb_test "show arch" ".*powerpc:common.*" "checking for PPC arch"]} {
-    return -1
+    return
 }
 
 gdb_test "next" ""
 
 for {set i 0} {$i < 16} {incr i 1} {
diff --git a/gdb/testsuite/gdb.arch/powerpc-fpscr-gcore.exp b/gdb/testsuite/gdb.arch/powerpc-fpscr-gcore.exp
index e0afcf419272..b2ead4cef5ee 100644
--- a/gdb/testsuite/gdb.arch/powerpc-fpscr-gcore.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-fpscr-gcore.exp
@@ -29,17 +29,17 @@  gdb_produce_source $gen_src {
 	    return 0;
 	}
 }
 
 if {[build_executable "compile" $binfile $gen_src] == -1} {
-    return -1
+    return
 }
 
 clean_restart $testfile
 
 if {![runto_main]} {
-    return -1
+    return
 }
 
 # Check if our target has FPSCR
 proc check_fpscr_access {} {
     global gdb_prompt
@@ -57,28 +57,28 @@  proc check_fpscr_access {} {
     }
     return 0
 }
 
 if { ![check_fpscr_access] } {
-    return -1
+    return
 }
 
 set fpscr_size_process [get_integer_valueof "sizeof(\$fpscr)" -1 "process size"]
 
 set core_filename [standard_output_file "$testfile.core"]
 set core_generated [gdb_gcore_cmd "$core_filename" "generate core file"]
 
 if { !$core_generated } {
-    return -1
+    return
 }
 
 clean_restart
 
 set core_loaded [gdb_core_cmd "$core_filename" "load core file"]
 
 if { $core_loaded != 1 } {
-    return -1
+    return
 }
 
 set fpscr_size_corefile [get_integer_valueof "sizeof(\$fpscr)" -2 "corefile size" ]
 
 set test "fpscr size matches"
diff --git a/gdb/testsuite/gdb.arch/powerpc-lnia.exp b/gdb/testsuite/gdb.arch/powerpc-lnia.exp
index 799b6fc06b79..700ba0dea67d 100644
--- a/gdb/testsuite/gdb.arch/powerpc-lnia.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-lnia.exp
@@ -29,11 +29,11 @@  set retval 0
 
 standard_testfile .s
 
 if { [prepare_for_testing "failed to prepare" $testfile "$srcfile" \
       {debug quiet}] } {
-    return -1
+    return
 }
 
 if {![runto_main]} {
       return
 }
diff --git a/gdb/testsuite/gdb.arch/powerpc-plxv-nonrel.exp b/gdb/testsuite/gdb.arch/powerpc-plxv-nonrel.exp
index 4f90080cf564..865ceb18ccd1 100644
--- a/gdb/testsuite/gdb.arch/powerpc-plxv-nonrel.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-plxv-nonrel.exp
@@ -22,11 +22,11 @@  set retval 0
 
 standard_testfile .s
 
 if { [prepare_for_testing "failed to prepare" $testfile "$srcfile" \
       {debug quiet}] } {
-    return -1
+    return
 }
 
 gdb_test "set radix 0b10000"
 gdb_test "set debug displaced"
 
diff --git a/gdb/testsuite/gdb.arch/powerpc-power10.exp b/gdb/testsuite/gdb.arch/powerpc-power10.exp
index 4fa55b01da1b..1389a5568e27 100644
--- a/gdb/testsuite/gdb.arch/powerpc-power10.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-power10.exp
@@ -22,11 +22,11 @@  set objfile [standard_output_file ${testfile}.o]
 
 require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
 gdb_load $objfile
 
diff --git a/gdb/testsuite/gdb.arch/powerpc-power7.exp b/gdb/testsuite/gdb.arch/powerpc-power7.exp
index 17ef875f4aef..810f536f8b0f 100644
--- a/gdb/testsuite/gdb.arch/powerpc-power7.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-power7.exp
@@ -22,11 +22,11 @@  set objfile [standard_output_file ${testfile}.o]
 
 require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
 gdb_load $objfile
 
diff --git a/gdb/testsuite/gdb.arch/powerpc-power8.exp b/gdb/testsuite/gdb.arch/powerpc-power8.exp
index baefb3cb6554..14a49b71f987 100644
--- a/gdb/testsuite/gdb.arch/powerpc-power8.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-power8.exp
@@ -22,11 +22,11 @@  set objfile [standard_output_file ${testfile}.o]
 
 require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
 gdb_load $objfile
 
diff --git a/gdb/testsuite/gdb.arch/powerpc-power9.exp b/gdb/testsuite/gdb.arch/powerpc-power9.exp
index 7e446330e68a..2d4884ce815f 100644
--- a/gdb/testsuite/gdb.arch/powerpc-power9.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-power9.exp
@@ -22,11 +22,11 @@  set objfile [standard_output_file ${testfile}.o]
 
 require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
 gdb_load $objfile
 
diff --git a/gdb/testsuite/gdb.arch/powerpc-prologue-frame.exp b/gdb/testsuite/gdb.arch/powerpc-prologue-frame.exp
index 1a6c4ca367e3..1f41f3aaaa3f 100644
--- a/gdb/testsuite/gdb.arch/powerpc-prologue-frame.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-prologue-frame.exp
@@ -20,11 +20,11 @@  set binfile [standard_output_file ${testfile}]
 
 if {[gdb_compile \
       [list ${srcdir}/${subdir}/$srcfile ${srcdir}/${subdir}/$srcfile2] \
       "${binfile}" executable {}] != ""} {
     untested "failed to build $binfile"
-    return -1
+    return
 }
 
 
 clean_restart $testfile
 
diff --git a/gdb/testsuite/gdb.arch/powerpc-prologue.exp b/gdb/testsuite/gdb.arch/powerpc-prologue.exp
index c52507a716c9..7256f7514649 100644
--- a/gdb/testsuite/gdb.arch/powerpc-prologue.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-prologue.exp
@@ -25,22 +25,22 @@  set binfile [standard_output_file ${testfile}]
 
 # Don't use "debug", so that we don't have line information for the assembly
 # fragments.
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {}] != "" } {
     untested "PowerPC prologue tests"
-    return -1
+    return
 }
 
 
 clean_restart $testfile
 
 #
 # Run to `main' where we begin our tests.
 #
 
 if {![runto_main]} {
-    return 0
+    return
 }
 
 # Testcase for PIC prologue.
 
 gdb_breakpoint "gdb2029_marker"
diff --git a/gdb/testsuite/gdb.arch/powerpc-stackless.exp b/gdb/testsuite/gdb.arch/powerpc-stackless.exp
index 78084682d51c..eff840886079 100644
--- a/gdb/testsuite/gdb.arch/powerpc-stackless.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-stackless.exp
@@ -18,11 +18,11 @@ 
 require {istarget "powerpc*-*-*"}
 
 standard_testfile .S
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 # Run until SIGSEGV.
 gdb_run_cmd
 
diff --git a/gdb/testsuite/gdb.arch/powerpc-tar.exp b/gdb/testsuite/gdb.arch/powerpc-tar.exp
index dadafdfcebc9..0bda11caf9ca 100644
--- a/gdb/testsuite/gdb.arch/powerpc-tar.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-tar.exp
@@ -25,11 +25,11 @@ 
 require {istarget "powerpc*-*-linux*"}
 
 standard_testfile .c
 
 if {[build_executable "compile" $binfile $srcfile {debug}] == -1} {
-    return -1
+    return
 }
 
 proc check_register_access { regname } {
     global gdb_prompt
 
diff --git a/gdb/testsuite/gdb.arch/powerpc-trap.exp b/gdb/testsuite/gdb.arch/powerpc-trap.exp
index 5d94efa9db41..099ae8da0a9d 100644
--- a/gdb/testsuite/gdb.arch/powerpc-trap.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-trap.exp
@@ -32,16 +32,16 @@  if { [istarget powerpc-*] } {
     unsupported "Skipping powerpc-specific tests"
     return
 }
 
 if {[prepare_for_testing "failed to prepare" ${testfile} ${srcfile}]} {
-    return -1
+    return
 }
 
 if {![runto_main]} {
     untested "could not run to main"
-    return -1
+    return
 }
 
 set keep_going 1
 set count 0
 
diff --git a/gdb/testsuite/gdb.arch/powerpc-vsx-gcore.exp b/gdb/testsuite/gdb.arch/powerpc-vsx-gcore.exp
index b0c197f6a95f..3f51b0f7fff5 100644
--- a/gdb/testsuite/gdb.arch/powerpc-vsx-gcore.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-vsx-gcore.exp
@@ -29,17 +29,17 @@  gdb_produce_source $gen_src {
 	    return 0;
 	}
 }
 
 if {[build_executable "compile" $binfile $gen_src] == -1} {
-    return -1
+    return
 }
 
 clean_restart $testfile
 
 if {![runto_main]} {
-    return -1
+    return
 }
 
 # Check if VSX register access through gdb is supported
 proc check_vsx_access {} {
     global gdb_prompt
@@ -57,30 +57,30 @@  proc check_vsx_access {} {
     }
     return 0
 }
 
 if { ![check_vsx_access] } {
-    return -1
+    return
 }
 
 for {set i 0} {$i < 64} {incr i 1} {
     gdb_test_no_output "set \$vs$i.uint128 = $i"
 }
 
 set core_filename [standard_output_file "$testfile.core"]
 set core_generated [gdb_gcore_cmd "$core_filename" "generate core file"]
 
 if { !$core_generated } {
-    return -1
+    return
 }
 
 clean_restart
 
 set core_loaded [gdb_core_cmd "$core_filename" "load core file"]
 
 if { $core_loaded != 1 } {
-    return -1
+    return
 }
 
 for {set i 0} {$i < 64} {incr i 1} {
     gdb_test "print \$vs$i.uint128" ".* = $i" "print vs$i"
 }
diff --git a/gdb/testsuite/gdb.arch/powerpc-vsx.exp b/gdb/testsuite/gdb.arch/powerpc-vsx.exp
index aeefe3df0746..c53502ce332c 100644
--- a/gdb/testsuite/gdb.arch/powerpc-vsx.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-vsx.exp
@@ -22,11 +22,11 @@  set objfile [standard_output_file ${testfile}.o]
 
 require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
 gdb_load $objfile
 
diff --git a/gdb/testsuite/gdb.arch/powerpc-vsx2.exp b/gdb/testsuite/gdb.arch/powerpc-vsx2.exp
index eb27085d7339..23ff53d855a3 100644
--- a/gdb/testsuite/gdb.arch/powerpc-vsx2.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-vsx2.exp
@@ -22,11 +22,11 @@  set objfile [standard_output_file ${testfile}.o]
 
 require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
 gdb_load $objfile
 
diff --git a/gdb/testsuite/gdb.arch/powerpc-vsx3.exp b/gdb/testsuite/gdb.arch/powerpc-vsx3.exp
index 9e9661b39df1..89e9775962bd 100644
--- a/gdb/testsuite/gdb.arch/powerpc-vsx3.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-vsx3.exp
@@ -22,11 +22,11 @@  set objfile [standard_output_file ${testfile}.o]
 
 require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
 gdb_load $objfile
 
diff --git a/gdb/testsuite/gdb.arch/powerpc64-prologue.exp b/gdb/testsuite/gdb.arch/powerpc64-prologue.exp
index 1add4a4d2929..342d18ea4bd8 100644
--- a/gdb/testsuite/gdb.arch/powerpc64-prologue.exp
+++ b/gdb/testsuite/gdb.arch/powerpc64-prologue.exp
@@ -23,21 +23,21 @@  standard_testfile .c
 
 # Don't use "debug", so that we don't have line information for the assembly
 # fragments.
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {}] != "" } {
     untested "PowerPC prologue tests"
-    return -1
+    return
 }
 
 if { [prepare_for_testing "failed to prepare" $testfile "$srcfile" \
       {}] } {
-    return -1
+    return
 }
 
 # Run to `main' where we begin our tests.
 if {![runto_main]} {
-    return 0
+    return
 }
 
 # Testcase for PIC prologue.
 gdb_breakpoint "gdb2029_marker"
 gdb_test "continue" "Breakpoint $decimal, $hex in gdb2029_marker \\(\\).*" \
diff --git a/gdb/testsuite/gdb.arch/ppc-dfp.exp b/gdb/testsuite/gdb.arch/ppc-dfp.exp
index 08c732774df1..becfc8206ca9 100644
--- a/gdb/testsuite/gdb.arch/ppc-dfp.exp
+++ b/gdb/testsuite/gdb.arch/ppc-dfp.exp
@@ -21,16 +21,16 @@  require {istarget "powerpc*"}
 standard_testfile
 
 if {![test_compiler_info gcc*]} {
     # We use GCC's extended asm syntax
     warning "unknown compiler"
-    return -1
+    return
 }
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {quiet debug}] != "" } {
     unsupported "this machine doesn't support Decimal Floating Point."
-    return -1
+    return
 }
 
 clean_restart $::testfile
 
 gdb_breakpoint [gdb_get_line_number "Set DFP rounding mode."]
@@ -40,11 +40,11 @@  gdb_run_cmd
 # When the prompt comes back we'll be at the Set DFP rounding mode breakpoint.
 # Unless the program bails out after checking AT_HWCAP.
 gdb_expect {
   -re "$inferior_exited_re with code 01.\[\r\n\]+$gdb_prompt $" {
     unsupported "this machine doesn't support Decimal Floating Point."
-    return -1
+    return
   }
 
   -re ".*$gdb_prompt $" {}
 }
 
diff --git a/gdb/testsuite/gdb.arch/ppc-fp.exp b/gdb/testsuite/gdb.arch/ppc-fp.exp
index e935b5ab1882..f17e37102ad5 100644
--- a/gdb/testsuite/gdb.arch/ppc-fp.exp
+++ b/gdb/testsuite/gdb.arch/ppc-fp.exp
@@ -21,16 +21,16 @@  require {istarget "powerpc*"}
 standard_testfile
 
 if {![test_compiler_info gcc*]} {
     # We use GCC's extended asm syntax
     warning "unknown compiler"
-    return -1
+    return
 }
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {quiet debug}] != "" } {
     verbose "Skipping FPSCR tests."
-    return -1
+    return
 }
 
 clean_restart $::testfile
 
 gdb_breakpoint [gdb_get_line_number "Invalid operation."]
diff --git a/gdb/testsuite/gdb.arch/ppc64-atomic-inst.exp b/gdb/testsuite/gdb.arch/ppc64-atomic-inst.exp
index c020df807f44..56854c75394d 100644
--- a/gdb/testsuite/gdb.arch/ppc64-atomic-inst.exp
+++ b/gdb/testsuite/gdb.arch/ppc64-atomic-inst.exp
@@ -22,11 +22,11 @@ 
 require {istarget "powerpc*"} is_lp64_target
 
 standard_testfile .S
 
 if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile} {debug quiet}] } {
-    return -1
+    return
 }
 
 # The test proper.  DISPLACED is true if we should try with displaced
 # stepping.
 proc do_test { displaced } {
diff --git a/gdb/testsuite/gdb.arch/ppc64-break-on-_exit.exp b/gdb/testsuite/gdb.arch/ppc64-break-on-_exit.exp
index fd8eae300c34..684fa04213de 100644
--- a/gdb/testsuite/gdb.arch/ppc64-break-on-_exit.exp
+++ b/gdb/testsuite/gdb.arch/ppc64-break-on-_exit.exp
@@ -37,15 +37,15 @@  if {[info exists COMPILE]} {
     standard_testfile .s -main.c
 }
 
 if { [prepare_for_testing "failed to prepare" ${testfile} \
 	  [list $srcfile $srcfile2] $flags] } {
-    return -1
+    return
 }
 
 if {![runto_main]} {
-    return 0
+    return
 }
 
 gdb_breakpoint "_exit"
 
 # If the skip_prologue analysis of _exit is too eager, we may not hit the
diff --git a/gdb/testsuite/gdb.arch/ppc64-isa207-atomic-inst.exp b/gdb/testsuite/gdb.arch/ppc64-isa207-atomic-inst.exp
index 482dbb4d04fd..db8c7582d0c3 100644
--- a/gdb/testsuite/gdb.arch/ppc64-isa207-atomic-inst.exp
+++ b/gdb/testsuite/gdb.arch/ppc64-isa207-atomic-inst.exp
@@ -25,11 +25,11 @@  require {istarget "powerpc*"} is_lp64_target
 
 standard_testfile  .c .S
 
 if { [prepare_for_testing "failed to prepare" $testfile "$srcfile $srcfile2" \
       {debug quiet}] } {
-    return -1
+    return
 }
 
 # The test proper.  DISPLACED is true if we should try with displaced
 # stepping.
 
diff --git a/gdb/testsuite/gdb.arch/ppc64-symtab-cordic.exp b/gdb/testsuite/gdb.arch/ppc64-symtab-cordic.exp
index 9a804591514d..c418435921b6 100644
--- a/gdb/testsuite/gdb.arch/ppc64-symtab-cordic.exp
+++ b/gdb/testsuite/gdb.arch/ppc64-symtab-cordic.exp
@@ -27,17 +27,17 @@  set kobz2file ${srcdir}/${subdir}/cordic.ko.bz2
 set kodebugbz2file ${srcdir}/${subdir}/cordic.ko.debug.bz2
 
 set kofile [decompress_bz2 $kobz2file]
 if { $kofile == "" } {
     untested "failed bzip2 for [file tail $kobz2file]"
-    return -1
+    return
 }
 
 set kodebugfile [decompress_bz2 $kodebugbz2file]
 if { $kodebugfile == "" } {
     untested "failed bzip2 for [file tail $kodebugbz2file]"
-    return -1
+    return
 }
 
 # This test won't work properly if system debuginfo is installed.
 # Test message is suppressed by "" as otherwise we could print PASS+UNTESTED
 # result to gdb.sum making a false feeling the issue has been tested.
diff --git a/gdb/testsuite/gdb.arch/vsx-regs.exp b/gdb/testsuite/gdb.arch/vsx-regs.exp
index cac7be5ee09e..bb5675f17542 100644
--- a/gdb/testsuite/gdb.arch/vsx-regs.exp
+++ b/gdb/testsuite/gdb.arch/vsx-regs.exp
@@ -28,11 +28,11 @@  if {[test_compiler_info gcc*]} {
     set compile_flags "$compile_flags additional_flags=-maltivec additional_flags=-mabi=altivec"
 } elseif {[test_compiler_info xlc*]} {
     set compile_flags "$compile_flags additional_flags=-qaltivec"
 } else {
     warning "unknown compiler"
-    return -1
+    return
 }
 
 if  { [gdb_compile ${srcdir}/${subdir}/${srcfile} ${binfile} executable $compile_flags] != "" } {
      untested "failed to compile"
      return -1
@@ -43,11 +43,11 @@  gdb_reinitialize_dir $srcdir/$subdir
 gdb_load ${binfile}
 
 # Run to `main' where we begin our tests.
 
 if {![runto_main]} {
-    return 0
+    return
 }
 
 set endianness [get_endianness]
 
 # Data sets used throughout the test
diff --git a/gdb/testsuite/gdb.arch/vsx-vsr-float28.exp b/gdb/testsuite/gdb.arch/vsx-vsr-float28.exp
index 9c44cdd61637..217f240f437d 100644
--- a/gdb/testsuite/gdb.arch/vsx-vsr-float28.exp
+++ b/gdb/testsuite/gdb.arch/vsx-vsr-float28.exp
@@ -27,11 +27,11 @@  if {[test_compiler_info gcc*]} {
     set compile_flags "$compile_flags additional_flags=-maltivec additional_flags=-mabi=altivec"
 } elseif {[test_compiler_info xlc*]} {
     set compile_flags "$compile_flags additional_flags=-qaltivec"
 } else {
     warning "unknown compiler"
-    return -1
+    return
 }
 
 if  { [gdb_compile ${srcdir}/${subdir}/${srcfile} ${binfile} executable $compile_flags] != "" } {
      untested "failed to compile"
      return -1
@@ -42,11 +42,11 @@  gdb_reinitialize_dir $srcdir/$subdir
 gdb_load ${binfile}
 
 # Run to `main' where we begin our tests.
 
 if {![runto_main]} {
-    return 0
+    return
 }
 
 set endianness [get_endianness]
 
 # Data sets used throughout the test