]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
GDB: testsuite: powerpc: Don't return value from top-level (sed)
authorThiago Jung Bauermann <thiago.bauermann@linaro.org>
Wed, 29 Apr 2026 23:03:56 +0000 (20:03 -0300)
committerThiago Jung Bauermann <thiago.bauermann@linaro.org>
Fri, 8 May 2026 21:49:23 +0000 (21:49 +0000)
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.

Approved-By: Tom Tromey <tom@tromey.com>
37 files changed:
gdb/testsuite/gdb.arch/aix-sighandle.exp
gdb/testsuite/gdb.arch/altivec-abi.exp
gdb/testsuite/gdb.arch/altivec-regs.exp
gdb/testsuite/gdb.arch/e500-abi.exp
gdb/testsuite/gdb.arch/e500-prologue.exp
gdb/testsuite/gdb.arch/e500-regs.exp
gdb/testsuite/gdb.arch/powerpc-addpcis.exp
gdb/testsuite/gdb.arch/powerpc-aix-prologue.exp
gdb/testsuite/gdb.arch/powerpc-altivec.exp
gdb/testsuite/gdb.arch/powerpc-altivec2.exp
gdb/testsuite/gdb.arch/powerpc-altivec3.exp
gdb/testsuite/gdb.arch/powerpc-d128-regs.exp
gdb/testsuite/gdb.arch/powerpc-fpscr-gcore.exp
gdb/testsuite/gdb.arch/powerpc-lnia.exp
gdb/testsuite/gdb.arch/powerpc-plxv-nonrel.exp
gdb/testsuite/gdb.arch/powerpc-power10.exp
gdb/testsuite/gdb.arch/powerpc-power7.exp
gdb/testsuite/gdb.arch/powerpc-power8.exp
gdb/testsuite/gdb.arch/powerpc-power9.exp
gdb/testsuite/gdb.arch/powerpc-prologue-frame.exp
gdb/testsuite/gdb.arch/powerpc-prologue.exp
gdb/testsuite/gdb.arch/powerpc-stackless.exp
gdb/testsuite/gdb.arch/powerpc-tar.exp
gdb/testsuite/gdb.arch/powerpc-trap.exp
gdb/testsuite/gdb.arch/powerpc-vsx-gcore.exp
gdb/testsuite/gdb.arch/powerpc-vsx.exp
gdb/testsuite/gdb.arch/powerpc-vsx2.exp
gdb/testsuite/gdb.arch/powerpc-vsx3.exp
gdb/testsuite/gdb.arch/powerpc64-prologue.exp
gdb/testsuite/gdb.arch/ppc-dfp.exp
gdb/testsuite/gdb.arch/ppc-fp.exp
gdb/testsuite/gdb.arch/ppc64-atomic-inst.exp
gdb/testsuite/gdb.arch/ppc64-break-on-_exit.exp
gdb/testsuite/gdb.arch/ppc64-isa207-atomic-inst.exp
gdb/testsuite/gdb.arch/ppc64-symtab-cordic.exp
gdb/testsuite/gdb.arch/vsx-regs.exp
gdb/testsuite/gdb.arch/vsx-vsr-float28.exp

index df988b26a44da275930887bf6f7947eff88937d3..4e5bb50c85605b1e6507384ff19935637de5c0d7 100644 (file)
@@ -16,7 +16,7 @@
 require {istarget "powerpc*-*-aix*"}
 
 if { [prepare_for_testing "failed to prepare" aix-sighandle aix-sighandle.c] } {
-    return -1
+    return
 }
 
 set srcfile aix-sighandle.c
index 40b1c3519e284b7d8853a8474b23c5ce87f92115..f7bea3de4cedc97940dc0d543ae5e7f5ccdf325d 100644 (file)
@@ -170,5 +170,5 @@ if {[test_compiler_info gcc*]} {
     altivec_abi_tests "additional_flags=-qaltivec" "auto"
 } else {
     warning "unknown compiler"
-    return -1
+    return
 }
index 70a15a9190a861fbdadd506362f6e17ffe0e6f0f..4dd8610d8f152c16753edcb65d13a04ee5d58953 100644 (file)
@@ -32,7 +32,7 @@ if {[test_compiler_info gcc*]} {
     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] != "" } {
@@ -49,7 +49,7 @@ gdb_load ${binfile}
 #
 
 if {![runto_main]} {
-    return 0
+    return
 }
 
 gdb_breakpoint [gdb_get_line_number "start here"]
index 1c074f10aa7e7af765cf85e890599e774cfc9d74..6f549539e2c32e945785b3d1700596b623e67ff7 100644 (file)
@@ -42,7 +42,7 @@ gdb_load ${binfile}
 #
 
 if {![runto_main]} {
-    return 0
+    return
 }
 
 gdb_test "b marker" "Breakpoint 2 at.*file.*e500-abi.c, line ${::decimal}." "break marker"
index b3d0056fd97c7b3ed8b7b1e7ac785402bdf227e1..f6884285e3a3844a72c1a05ad8a2b98a31f6ac4f 100644 (file)
@@ -24,7 +24,7 @@ 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
index 2dc1ade4b448d1e70524b9f4b7cbc20d0f8ec259..b9d172c058d78eb58be0e2842e9908e646b21aa9 100644 (file)
@@ -41,7 +41,7 @@ gdb_load ${binfile}
 #
 
 if {![runto_main]} {
-    return 0
+    return
 }
 
 # set all the registers integer portions to 1
@@ -139,7 +139,7 @@ gdb_expect_list "info vector" ".*$gdb_prompt $" {
 clean_restart
 gdb_load $binfile
 if {![runto_main]} {
-    return 0
+    return
 }
 
 gdb_test "break vector_fun" \
index e699e730021c7bf7b92227e061e7dbe311a13fae..a8895446cde362430910cc244d0068f6096561d7 100644 (file)
@@ -32,7 +32,7 @@ standard_testfile .s
 
 if { [prepare_for_testing "failed to prepare" $testfile "$srcfile" \
       {debug quiet}] } {
-    return -1
+    return
 }
 
 if {![runto_main]} {
index f465f36fe2dad8d95bb7c9d5dc1408717d19ae2d..0da2383b1082b5ba528e47762077abca416b1f43 100644 (file)
@@ -25,7 +25,7 @@ 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
index edbb822ff8382621eb0ab908bdb5aa5dea908c08..cb89ba7e518ff991333166e493fbfd4c191aef56 100644 (file)
@@ -24,7 +24,7 @@ require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
index 1b34cb6a8a332b2889ddd6ef3291328fb82dde5b..621f44dc4e429dfa152b47a01dfdd33b60ee8919 100644 (file)
@@ -24,7 +24,7 @@ require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
index 602e65217494faea896172acaf9c43bf5893742a..6607e24d7a8fae27a92ae7c554f6cb885973e097 100644 (file)
@@ -24,7 +24,7 @@ require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
index 05ae31877e1b8b98509d9716279f9820d752a045..10c77ef29489f5313bdec39694a5389071919b28 100644 (file)
@@ -36,7 +36,7 @@ if {![runto_main]} {
 }
 
 if {[gdb_test "show arch" ".*powerpc:common.*" "checking for PPC arch"]} {
-    return -1
+    return
 }
 
 gdb_test "next" ""
index e0afcf419272905d88a7aedc6ce91ebd960eb6eb..b2ead4cef5ee3402111595a09573f5ab92ac2d2c 100644 (file)
@@ -31,13 +31,13 @@ gdb_produce_source $gen_src {
 }
 
 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
@@ -59,7 +59,7 @@ proc check_fpscr_access {} {
 }
 
 if { ![check_fpscr_access] } {
-    return -1
+    return
 }
 
 set fpscr_size_process [get_integer_valueof "sizeof(\$fpscr)" -1 "process size"]
@@ -68,7 +68,7 @@ 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
@@ -76,7 +76,7 @@ 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" ]
index 799b6fc06b79adaf7f0e48fb00c231c30788c883..700ba0dea67defcb0e0111f0dbd310eb66fe4769 100644 (file)
@@ -31,7 +31,7 @@ standard_testfile .s
 
 if { [prepare_for_testing "failed to prepare" $testfile "$srcfile" \
       {debug quiet}] } {
-    return -1
+    return
 }
 
 if {![runto_main]} {
index 4f90080cf5642de2a896d50ac887fdc6ce68e586..865ceb18ccd198fa87d65cc740ef4ff0453db86c 100644 (file)
@@ -24,7 +24,7 @@ standard_testfile .s
 
 if { [prepare_for_testing "failed to prepare" $testfile "$srcfile" \
       {debug quiet}] } {
-    return -1
+    return
 }
 
 gdb_test "set radix 0b10000"
index 4fa55b01da1be6b1b5958b2ae363c59830bbcf3f..1389a5568e27cacf67c91104c9d7c2ffffe508c9 100644 (file)
@@ -24,7 +24,7 @@ require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
index 17ef875f4aef0a6661b2983888bf2cc8c2d28059..810f536f8b0fc90c56674463f0cc4e0c057b1a7c 100644 (file)
@@ -24,7 +24,7 @@ require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
index baefb3cb6554868843b7b93c99f2850e5b903947..14a49b71f9877c31d738cd797ef0e160e87b6dfb 100644 (file)
@@ -24,7 +24,7 @@ require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
index 7e446330e68a6b64eff2dfc6ecad47c6531870ab..2d4884ce815fd16e0370dac5360fe99ea3c699b2 100644 (file)
@@ -24,7 +24,7 @@ require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
index 1a6c4ca367e3ca1f526f8decde963391dcb4bbf6..1f41f3aaaa3f69096ef81dc9b308285ed7c54712 100644 (file)
@@ -22,7 +22,7 @@ if {[gdb_compile \
       [list ${srcdir}/${subdir}/$srcfile ${srcdir}/${subdir}/$srcfile2] \
       "${binfile}" executable {}] != ""} {
     untested "failed to build $binfile"
-    return -1
+    return
 }
 
 
index c52507a716c96c436656a7911fdeb7595ca875c0..7256f751464961ef074f5569e319cffcb2616b17 100644 (file)
@@ -27,7 +27,7 @@ set binfile [standard_output_file ${testfile}]
 # fragments.
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {}] != "" } {
     untested "PowerPC prologue tests"
-    return -1
+    return
 }
 
 
@@ -38,7 +38,7 @@ clean_restart $testfile
 #
 
 if {![runto_main]} {
-    return 0
+    return
 }
 
 # Testcase for PIC prologue.
index 78084682d51c36ac73c302b2fdc4d40dea5fa763..eff840886079d30b464ecd0b06fd19141f4c85cd 100644 (file)
@@ -20,7 +20,7 @@ require {istarget "powerpc*-*-*"}
 standard_testfile .S
 
 if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
-    return -1
+    return
 }
 
 # Run until SIGSEGV.
index dadafdfcebc9aa25411811576f4c68e3c3270197..0bda11caf9ca7240a030cdd05499449372ee5e4b 100644 (file)
@@ -27,7 +27,7 @@ require {istarget "powerpc*-*-linux*"}
 standard_testfile .c
 
 if {[build_executable "compile" $binfile $srcfile {debug}] == -1} {
-    return -1
+    return
 }
 
 proc check_register_access { regname } {
index 5d94efa9db414c6b33162fe21a4ddcb9b6f1ad29..099ae8da0a9d3040e45b05833f704ceefc8032eb 100644 (file)
@@ -34,12 +34,12 @@ if { [istarget powerpc-*] } {
 }
 
 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
index b0c197f6a95f4ed9fbdf849ed6d748f0dafebea8..3f51b0f7fff58309b96e0abc58be826e6dabf746 100644 (file)
@@ -31,13 +31,13 @@ gdb_produce_source $gen_src {
 }
 
 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
@@ -59,7 +59,7 @@ proc check_vsx_access {} {
 }
 
 if { ![check_vsx_access] } {
-    return -1
+    return
 }
 
 for {set i 0} {$i < 64} {incr i 1} {
@@ -70,7 +70,7 @@ 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
@@ -78,7 +78,7 @@ 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} {
index aeefe3df07463da5e0dffcf69442762507d954c7..c53502ce332c3bcd3261ff20102b419672033637 100644 (file)
@@ -24,7 +24,7 @@ require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
index eb27085d7339527d7331386a2f538fc058529647..23ff53d855a3b8fa1fcffc7634cb6b4bee618393 100644 (file)
@@ -24,7 +24,7 @@ require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
index 9e9661b39df17605ff317866fda058c33e195d5d..89e9775962bd92f8b2b1e2ae14f47d0c3f80bb99 100644 (file)
@@ -24,7 +24,7 @@ require {istarget "powerpc*-*-*"}
 
 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${objfile}" object {debug}] != "" } {
     untested "PowerPC instructions disassembly"
-    return -1
+    return
 }
 
 clean_restart
index 1add4a4d2929dffc3dfe5240761021344734c1e9..342d18ea4bd873eeb61067ccf38ff03564fee495 100644 (file)
@@ -25,17 +25,17 @@ standard_testfile .c
 # 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.
index 08c732774df1e48a7236f8efd5afedac986bbd39..becfc8206ca98d3c183e53c27249f884dd29a8c1 100644 (file)
@@ -23,12 +23,12 @@ 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
@@ -42,7 +42,7 @@ gdb_run_cmd
 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 $" {}
index e935b5ab1882a83302a4586a4d1e7c8b75aeb402..f17e37102ad53c0cfa4ebe91fe71ca4fc959824b 100644 (file)
@@ -23,12 +23,12 @@ 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
index c020df807f447d8c46f9c73fcb1199b2ea2ab8cd..56854c75394d79de2dfec3696caf5cc6cad07ecd 100644 (file)
@@ -24,7 +24,7 @@ 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
index fd8eae300c34fa6f35cde48ada23c8f1eb45e08e..684fa04213de23ed39ac612cc30cb419f72a51f6 100644 (file)
@@ -39,11 +39,11 @@ if {[info exists COMPILE]} {
 
 if { [prepare_for_testing "failed to prepare" ${testfile} \
          [list $srcfile $srcfile2] $flags] } {
-    return -1
+    return
 }
 
 if {![runto_main]} {
-    return 0
+    return
 }
 
 gdb_breakpoint "_exit"
index 482dbb4d04fd774da51c498cd3dd79df177d565c..db8c7582d0c3b0efba5368202d5b8d3b15604943 100644 (file)
@@ -27,7 +27,7 @@ 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
index 9a804591514de0f6351c1f02942113c95ea49eb8..c418435921b602bac689a19b8b16f509cb8dadf2 100644 (file)
@@ -29,13 +29,13 @@ 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.
index cac7be5ee09eda9093ee78de0e92fef05a945f03..bb5675f175421dfa3ad4598d90beee961787d431 100644 (file)
@@ -30,7 +30,7 @@ if {[test_compiler_info gcc*]} {
     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] != "" } {
@@ -45,7 +45,7 @@ gdb_load ${binfile}
 # Run to `main' where we begin our tests.
 
 if {![runto_main]} {
-    return 0
+    return
 }
 
 set endianness [get_endianness]
index 9c44cdd6163746ab2bcdf5e522cc6a92318b1a6d..217f240f437d08b37bd2be26c663fa97c5fd7049 100644 (file)
@@ -29,7 +29,7 @@ if {[test_compiler_info gcc*]} {
     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] != "" } {
@@ -44,7 +44,7 @@ gdb_load ${binfile}
 # Run to `main' where we begin our tests.
 
 if {![runto_main]} {
-    return 0
+    return
 }
 
 set endianness [get_endianness]