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

Message ID 20260429230432.60487-7-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 manually changes the "return -1" statements that weren't
caught by the sed command because they were in a misaligned line, which
is then aligned by this patch.
---
 gdb/testsuite/gdb.arch/altivec-regs.exp           | 4 ++--
 gdb/testsuite/gdb.arch/e500-abi.exp               | 4 ++--
 gdb/testsuite/gdb.arch/e500-regs.exp              | 4 ++--
 gdb/testsuite/gdb.arch/powerpc-d128-regs.exp      | 4 ++--
 gdb/testsuite/gdb.arch/powerpc-prologue-frame.exp | 4 ++--
 gdb/testsuite/gdb.arch/vsx-regs.exp               | 6 +++---
 gdb/testsuite/gdb.arch/vsx-vsr-float28.exp        | 4 ++--
 7 files changed, 15 insertions(+), 15 deletions(-)
  

Patch

diff --git a/gdb/testsuite/gdb.arch/altivec-regs.exp b/gdb/testsuite/gdb.arch/altivec-regs.exp
index 4dd8610d8f15..04dc342de6d4 100644
--- a/gdb/testsuite/gdb.arch/altivec-regs.exp
+++ b/gdb/testsuite/gdb.arch/altivec-regs.exp
@@ -34,12 +34,12 @@  if {[test_compiler_info gcc*]} {
     warning "unknown compiler"
     return
 }
 
 if  { [gdb_compile ${srcdir}/${subdir}/${srcfile} ${binfile} executable $compile_flags] != "" } {
-     untested "failed to compile"
-     return -1
+    untested "failed to compile"
+    return
 }
 
 gdb_start
 gdb_reinitialize_dir $srcdir/$subdir
 gdb_load ${binfile}
diff --git a/gdb/testsuite/gdb.arch/e500-abi.exp b/gdb/testsuite/gdb.arch/e500-abi.exp
index 6f549539e2c3..e0223fd67888 100644
--- a/gdb/testsuite/gdb.arch/e500-abi.exp
+++ b/gdb/testsuite/gdb.arch/e500-abi.exp
@@ -27,12 +27,12 @@  set testfile "e500-abi"
 set binfile ${objdir}/${subdir}/${testfile}
 
 set src1 ${srcdir}/${subdir}/${testfile}.c
 
 if  { [gdb_compile ${src1} ${binfile} executable {debug nowarnings}] != "" } {
-     untested "failed to compile"
-     return -1
+    untested "failed to compile"
+    return
 }
 
 gdb_start
 gdb_reinitialize_dir $srcdir/$subdir
 gdb_load ${binfile}
diff --git a/gdb/testsuite/gdb.arch/e500-regs.exp b/gdb/testsuite/gdb.arch/e500-regs.exp
index b9d172c058d7..03d6f42b5332 100644
--- a/gdb/testsuite/gdb.arch/e500-regs.exp
+++ b/gdb/testsuite/gdb.arch/e500-regs.exp
@@ -26,12 +26,12 @@  require {istarget "powerpc-*eabispe"}
 set testfile "e500-regs"
 set binfile ${objdir}/${subdir}/${testfile}
 set src1 ${srcdir}/${subdir}/${testfile}.c
 
 if  { [gdb_compile ${src1} ${binfile} executable {debug nowarnings}] != "" } {
-     untested "failed to compile"
-     return -1
+    untested "failed to compile"
+    return
 }
 
 gdb_start
 gdb_reinitialize_dir $srcdir/$subdir
 gdb_load ${binfile}
diff --git a/gdb/testsuite/gdb.arch/powerpc-d128-regs.exp b/gdb/testsuite/gdb.arch/powerpc-d128-regs.exp
index 10c77ef29489..7043fac5e568 100644
--- a/gdb/testsuite/gdb.arch/powerpc-d128-regs.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-d128-regs.exp
@@ -23,12 +23,12 @@ 
 require {istarget "powerpc64*-*"}
 
 standard_testfile
 
 if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {quiet debug}] != "" } {
-     untested "failed to compile"
-     return -1
+    untested "failed to compile"
+    return
 }
 
 clean_restart $::testfile
 
 if {![runto_main]} {
diff --git a/gdb/testsuite/gdb.arch/powerpc-prologue-frame.exp b/gdb/testsuite/gdb.arch/powerpc-prologue-frame.exp
index 1f41f3aaaa3f..3899121f9341 100644
--- a/gdb/testsuite/gdb.arch/powerpc-prologue-frame.exp
+++ b/gdb/testsuite/gdb.arch/powerpc-prologue-frame.exp
@@ -27,12 +27,12 @@  if {[gdb_compile \
 
 
 clean_restart $testfile
 
 if {![runto bar]} {
-  untested "could not run to bar"
-  return -1
+    untested "could not run to bar"
+    return
 }
 
 gdb_test "bt" \
 	 "#0\[ \t\]*$hex in bar.*\r\n#1\[ \t\]*$hex in foo.*\r\n#2\[ \t\]*$hex in main.*" \
 	 "Backtrace to the main frame"
diff --git a/gdb/testsuite/gdb.arch/vsx-regs.exp b/gdb/testsuite/gdb.arch/vsx-regs.exp
index bb5675f17542..87f749f93b12 100644
--- a/gdb/testsuite/gdb.arch/vsx-regs.exp
+++ b/gdb/testsuite/gdb.arch/vsx-regs.exp
@@ -32,12 +32,12 @@  if {[test_compiler_info gcc*]} {
     warning "unknown compiler"
     return
 }
 
 if  { [gdb_compile ${srcdir}/${subdir}/${srcfile} ${binfile} executable $compile_flags] != "" } {
-     untested "failed to compile"
-     return -1
+    untested "failed to compile"
+    return
 }
 
 gdb_start
 gdb_reinitialize_dir $srcdir/$subdir
 gdb_load ${binfile}
@@ -179,11 +179,11 @@  for {set i 0} {$i < 32} {incr i 1} {
 }
 
 # Test reading the core file.
 
 if {!$core_supported} {
-  return -1
+    return
 }
 
 clean_restart
 gdb_load $binfile
 
diff --git a/gdb/testsuite/gdb.arch/vsx-vsr-float28.exp b/gdb/testsuite/gdb.arch/vsx-vsr-float28.exp
index 217f240f437d..436076f10038 100644
--- a/gdb/testsuite/gdb.arch/vsx-vsr-float28.exp
+++ b/gdb/testsuite/gdb.arch/vsx-vsr-float28.exp
@@ -31,12 +31,12 @@  if {[test_compiler_info gcc*]} {
     warning "unknown compiler"
     return
 }
 
 if  { [gdb_compile ${srcdir}/${subdir}/${srcfile} ${binfile} executable $compile_flags] != "" } {
-     untested "failed to compile"
-     return -1
+    untested "failed to compile"
+    return
 }
 
 gdb_start
 gdb_reinitialize_dir $srcdir/$subdir
 gdb_load ${binfile}