gdb/testsuite: remove use of then keyword from gdb.arch/*.exp
authorAndrew Burgess <aburgess@redhat.com>
Mon, 14 Nov 2022 11:39:14 +0000 (11:39 +0000)
committerAndrew Burgess <aburgess@redhat.com>
Mon, 28 Nov 2022 21:04:09 +0000 (21:04 +0000)
The canonical form of 'if' in modern TCL is 'if {} {}'.  But there's
still a bunch of places in the testsuite where we make use of the
'then' keyword, and sometimes these get copies into new tests, which
just spreads poor practice.

This commit removes all use of the 'then' keyword from the gdb.arch/
test script directory.

There should be no changes in what is tested after this commit.

94 files changed:
gdb/testsuite/gdb.arch/aarch64-unwind-pc.exp
gdb/testsuite/gdb.arch/alpha-step.exp
gdb/testsuite/gdb.arch/altivec-abi.exp
gdb/testsuite/gdb.arch/altivec-regs.exp
gdb/testsuite/gdb.arch/amd64-byte.exp
gdb/testsuite/gdb.arch/amd64-disp-step-avx.exp
gdb/testsuite/gdb.arch/amd64-disp-step.exp
gdb/testsuite/gdb.arch/amd64-dword.exp
gdb/testsuite/gdb.arch/amd64-gs_base.exp
gdb/testsuite/gdb.arch/amd64-init-x87-values.exp
gdb/testsuite/gdb.arch/amd64-word.exp
gdb/testsuite/gdb.arch/arc-analyze-prologue.exp
gdb/testsuite/gdb.arch/arc-decode-insn.exp
gdb/testsuite/gdb.arch/arc-disassembler-options.exp
gdb/testsuite/gdb.arch/arm-disassembler-options.exp
gdb/testsuite/gdb.arch/arm-disp-step.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/gdb1291.exp
gdb/testsuite/gdb.arch/gdb1431.exp
gdb/testsuite/gdb.arch/gdb1558.exp
gdb/testsuite/gdb.arch/i386-avx.exp
gdb/testsuite/gdb.arch/i386-biarch-core.exp
gdb/testsuite/gdb.arch/i386-bp_permanent.exp
gdb/testsuite/gdb.arch/i386-byte.exp
gdb/testsuite/gdb.arch/i386-disp-step.exp
gdb/testsuite/gdb.arch/i386-dr3-watch.exp
gdb/testsuite/gdb.arch/i386-float.exp
gdb/testsuite/gdb.arch/i386-gnu-cfi.exp
gdb/testsuite/gdb.arch/i386-permbkpt.exp
gdb/testsuite/gdb.arch/i386-prologue.exp
gdb/testsuite/gdb.arch/i386-signal.exp
gdb/testsuite/gdb.arch/i386-size-overlap.exp
gdb/testsuite/gdb.arch/i386-size.exp
gdb/testsuite/gdb.arch/i386-sse-stack-align.exp
gdb/testsuite/gdb.arch/i386-sse.exp
gdb/testsuite/gdb.arch/i386-unwind.exp
gdb/testsuite/gdb.arch/i386-word.exp
gdb/testsuite/gdb.arch/iwmmxt-regs.exp
gdb/testsuite/gdb.arch/mips-disassembler-options.exp
gdb/testsuite/gdb.arch/mips-fcr.exp
gdb/testsuite/gdb.arch/mips-fpregset-core.exp
gdb/testsuite/gdb.arch/mips16-thunks.exp
gdb/testsuite/gdb.arch/pa-nullify.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-disassembler-options.exp
gdb/testsuite/gdb.arch/powerpc-fpscr-gcore.exp
gdb/testsuite/gdb.arch/powerpc-htm-regs.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-ppr-dscr.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-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/ppc-longdouble.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/pr25124.exp
gdb/testsuite/gdb.arch/riscv-bp-infcall.exp
gdb/testsuite/gdb.arch/riscv-info-fcsr.exp
gdb/testsuite/gdb.arch/riscv-reg-aliases.exp
gdb/testsuite/gdb.arch/riscv-unwind-long-insn.exp
gdb/testsuite/gdb.arch/riscv64-unwind-prologue-with-ld-lw.exp
gdb/testsuite/gdb.arch/riscv64-unwind-prologue-with-mv.exp
gdb/testsuite/gdb.arch/s390-disassembler-options.exp
gdb/testsuite/gdb.arch/s390-stackless.exp
gdb/testsuite/gdb.arch/s390-tdbregs.exp
gdb/testsuite/gdb.arch/sparc-sysstep.exp
gdb/testsuite/gdb.arch/sparc64-adi.exp
gdb/testsuite/gdb.arch/sparc64-regs.exp
gdb/testsuite/gdb.arch/thumb-bx-pc.exp
gdb/testsuite/gdb.arch/thumb-prologue.exp
gdb/testsuite/gdb.arch/thumb-singlestep.exp
gdb/testsuite/gdb.arch/thumb2-it.exp
gdb/testsuite/gdb.arch/vsx-regs.exp
gdb/testsuite/gdb.arch/vsx-vsr-float28.exp

index 105b9a9cc3adbcbacb3634b6c556df5b8593b424..a6d549a9830563016064220437ce3aa13e8cd22c 100644 (file)
@@ -17,7 +17,7 @@
 
 # Test explicitly unwinding the PC DWARF register on aarch64
 
-if {![is_aarch64_target]} then {
+if {![is_aarch64_target]} {
     verbose "Skipping ${gdb_test_file_name}."
     return
 }
index b65ed8124b92d5bf6cbf6d63d7e3f8813f6bfe5d..2f046752c63b6acd31340f3da95957beafbb774b 100644 (file)
@@ -14,7 +14,7 @@
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
 
-if ![istarget "alpha*-*-*"] then {
+if {![istarget "alpha*-*-*"]} {
     verbose "Skipping alpha step tests."
     return
 }
@@ -37,7 +37,7 @@ proc test_stepi {function } {
     # Restart the program from scratch. If GDB got confused during one
     # of the previous tests, this makes sure that it doesn't affect
     # this series of tests.
-    if ![runto_main] then {
+    if {![runto_main]} {
         return 0
     }
 
index d46882ad7ac6e7074c2ba75d6fd52e4e569e5777..d32cac7169910ec6943be46e96792133980909a3 100644 (file)
@@ -21,7 +21,7 @@
 # This file uses altivec-abi.c for input.
 #
 
-if {![istarget "powerpc*"] || [skip_altivec_tests]} then {
+if {![istarget "powerpc*"] || [skip_altivec_tests]} {
     verbose "Skipping altivec abi tests."
     return
 }
@@ -56,7 +56,7 @@ proc altivec_abi_tests { extra_flags force_abi } {
     gdb_load ${binfile}
 
     # Run to `main' where we begin our tests.
-    if ![runto_main] then {
+    if {![runto_main]} {
        return -1
     }
 
index 42521a0a9abcd40ebf32f58979f8e8fc42e8b00f..43ef5ed25c878c71a99911a1f255be1b228d7c53 100644 (file)
@@ -22,7 +22,7 @@
 #
 
 
-if {![istarget "powerpc*"] || [skip_altivec_tests]} then {
+if {![istarget "powerpc*"] || [skip_altivec_tests]} {
     verbose "Skipping altivec register tests."
     return
 }
@@ -52,7 +52,7 @@ gdb_load ${binfile}
 # Run to `main' where we begin our tests.
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 085963230cc71d652f4ce15fa3c4a95d4e32a84c..8ab7b68bf6b65147d34212ee0304d801fc28969d 100644 (file)
@@ -33,7 +33,7 @@ if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable [list
 
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index a64c85bc0e910714a934152da28ee2068e29b610..1d79f274f732d311dbb59131c0a61e41cb9783fe 100644 (file)
@@ -42,7 +42,7 @@ if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile} $options] }
 gdb_test "set displaced-stepping on" ""
 gdb_test "show displaced-stepping" ".* displaced stepping .* is on.*"
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 7bd2d5fac5183dbbe25f1e26e303cf6272d8e2a8..eb303af1b09f9c925f24508f36bd9c51c2fc9f41 100644 (file)
@@ -35,7 +35,7 @@ if { [prepare_for_testing "failed to prepare" $testfile $srcfile $opts] } {
 gdb_test "set displaced-stepping on" ""
 gdb_test "show displaced-stepping" ".* displaced stepping .* is on.*"
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 0c297529dc0380c0fdf609e672083973152e6d94..5151a60da4ab39ed2587fe140bbfa53ffb8c96f2 100644 (file)
@@ -33,7 +33,7 @@ if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable [list
 
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 477e78eb842fc77633b654bba8769a7f9f7228c1..f0a1013f2b206ceba7b9da63590bc0cae8989d85 100644 (file)
@@ -15,7 +15,7 @@
 
 standard_testfile
 
-if { ![is_amd64_regs_target] } then {
+if {![is_amd64_regs_target]} {
     verbose "Untested x86_64 fs_base and gs_base tests."
     return
 }
index fdc633042b451a618e07d1d0b4ad339e8221d926..6a99afb472536fdda9dc07f158f9a78592903713 100644 (file)
@@ -18,7 +18,7 @@
 # Test initial values of x87 control registers, before any x87
 # instructions have been executed in the inferior.
 
-if ![is_amd64_regs_target] then {
+if {![is_amd64_regs_target]} {
     return
 }
 
@@ -45,7 +45,7 @@ proc_with_prefix check_x87_regs_around_init {} {
     clean_restart ${binfile}
 
     # Get things started.
-    if ![runto_main] then {
+    if {![runto_main]} {
        return 0
     }
 
@@ -119,7 +119,7 @@ proc_with_prefix check_setting_mxcsr_before_enable {} {
 
     clean_restart ${binfile}
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return 0
     }
 
@@ -153,7 +153,7 @@ proc_with_prefix check_setting_x87_regs_before_enable {} {
 
     clean_restart ${binfile}
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return 0
     }
 
index c518dc96c86dd6b8602673a430ad834f3e2712ea..51eac6620a1431a954210b034a537e9d0085fc76 100644 (file)
@@ -33,7 +33,7 @@ if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable [list
 
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index c7f2dfd9a39fd5af26970b42f5af523728205747..cf50bd9e3e61b0d0a9f60e6d2479553555026f62 100644 (file)
@@ -15,7 +15,7 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-if {![istarget "arc*-*-*"]} then {
+if {![istarget "arc*-*-*"]} {
     verbose "Skipping ARC prologue test."
     return
 }
index 5564373a49adb802609dc287be27a070b8701f61..182132bab8216ab920be105bebf73ec76183ffb4 100644 (file)
@@ -31,7 +31,7 @@
 # about requirements to actual test cases can be found in corresponding
 # assembly file of this test case (arc-decode-insn.S).
 
-if {![istarget "arc*-*-*"]} then {
+if {![istarget "arc*-*-*"]} {
     verbose "Skipping ARC decoder test."
     return
 }
index 86f4acad656ff798e8cd8d587248dc0599a58025..167359a0119db2b2bbff44f8d7bced40cbc0f7c7 100644 (file)
@@ -15,7 +15,7 @@
 
 # Test ARC disassembler options.
 
-if { ![istarget "arc-*-*"] } then {
+if {![istarget "arc-*-*"]} {
     verbose "Skipping ARC disassembler option test."
     return
 }
index 4e84ff9d4a7603bc707599535e98c59f71a75ed7..e562112badd130fcf0cdcc7e3b5c375f10024f61 100644 (file)
@@ -18,7 +18,7 @@
 # This test exercises set/show disassembler-options results are preserved
 # across multiple set architecture calls.
 
-if {![istarget "arm*-*-*"]} then {
+if {![istarget "arm*-*-*"]} {
     verbose "Skipping ARM disassembler options."
     return
 }
index e60ef7b1120ff5f03ae8ef7d5b4725a8efe041ce..ec6578257b3b1e8d4b2418ff96170ae453257dc2 100644 (file)
@@ -17,7 +17,7 @@
 
 # Test arm displaced stepping.
 
-if {![is_aarch32_target]} then {
+if {![is_aarch32_target]} {
     verbose "Skipping arm displaced stepping tests."
     return
 }
@@ -409,7 +409,7 @@ proc test_add_rn_pc {} {
 # which is in library might be compiled in Thumb.
 gdb_test_no_output "set displaced-stepping off"
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index cb5e46a68db72254794b9470daa289da5dc82a89..0a94d4346bce836722cbb6aebc82981fdd8f6452 100644 (file)
@@ -22,7 +22,7 @@
 #
 
 
-if ![istarget "powerpc-*eabispe"] then {
+if {![istarget "powerpc-*eabispe"]} {
     verbose "Skipping e500 abi tests."
     return
 }
@@ -45,7 +45,7 @@ gdb_load ${binfile}
 # Run to `main' where we begin our tests.
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 2887a83f23e029f8d9dad9f7b93f57aba25e464f..94b4a960e8ec152ea3be5f051f17730302740dac 100644 (file)
@@ -18,7 +18,7 @@
 # Test PowerPC E500 prologue analyzer.
 
 
-if ![istarget "powerpc-*"] then {
+if {![istarget "powerpc-*"]} {
     verbose "Skipping powerpc E500 prologue tests."
     return
 }
@@ -62,7 +62,7 @@ proc insert_breakpoint {function expected_location} {
     # instruction at that address (we're not interested in the insn
     # itself, but rather at the address printed at the begining of
     # the instruction).
-    if { $address != "" } then {
+    if {$address != ""} {
        gdb_test "x /i $address" \
                 ".*<$function\\+$expected_location>.*" \
                 "check $function breakpoint address"
index 254246d1a65c20f705e13365f01df941f222fc67..d5fd7656937f723a047957156663d7944d1cf903 100644 (file)
@@ -22,7 +22,7 @@
 #
 
 
-if ![istarget "powerpc-*eabispe"] then {
+if {![istarget "powerpc-*eabispe"]} {
     verbose "Skipping e500 register tests."
     return
 }
@@ -44,7 +44,7 @@ gdb_load ${binfile}
 # Run to `main' where we begin our tests.
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
@@ -144,7 +144,7 @@ gdb_exit
 gdb_start
 gdb_reinitialize_dir $srcdir/$subdir
 gdb_load ${binfile}
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index cc60ebfc3a1f4cf18e2b7c1ba0566cce5f2769f9..47487513a6e4d071ec39bb685c496387aee70ce0 100644 (file)
@@ -24,7 +24,7 @@
 # Test SH backtraces with >256 byte frame stack. (PR:1291)
 
 
-if ![istarget "sh-*-*"] then {
+if {![istarget "sh-*-*"]} {
     verbose "Skipping SH backtrace tests."
     return
 }
@@ -46,7 +46,7 @@ gdb_load ${binfile}
 # Run to `main' where we begin our tests.
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 78ff651df735301275fb1872775d90a07fb50760..d4585b38a39075b6faa2f6c6cacaef42058ea92b 100644 (file)
@@ -26,7 +26,7 @@
 # Observe that the until command doesn't go all the way to sub2.
 
 
-if ![istarget "sh-*-*"] then {
+if {![istarget "sh-*-*"]} {
     verbose "Skipping SH backtrace tests."
     return
 }
@@ -48,7 +48,7 @@ gdb_load ${binfile}
 # Run to `main' where we begin our tests.
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 13f92ddf6019583878baf31b6c3b79e338dfb9bd..a1094e221b2a4c509c5c931b1e7a3def12aa6424 100644 (file)
@@ -22,7 +22,7 @@
 # from main.
 
 
-if ![istarget "sh-*-*"] then {
+if {![istarget "sh-*-*"]} {
     verbose "Skipping SH breakpoint test."
     return
 }
index 950b411809e10332a2a74f22be95c45ebfa435c2..2a2c17115e7aa7d29ec0bf5f0e18c21bbba9d8ec 100644 (file)
@@ -48,7 +48,7 @@ if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable \
 
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
@@ -100,7 +100,7 @@ with_test_prefix "force-disable xml descriptions" {
            clean_restart ${binfile}
        }
 
-       if ![runto_main] then {
+       if {![runto_main]} {
            return
        }
 
index 95e60d0b55280230b9b20e60059fb44f521f2da2..bb1678a36d3dd79ca20b7d3e6737d56d6d5f67f3 100644 (file)
@@ -21,7 +21,7 @@
 # #define EM_X86_64       62              /* AMD x86-64 architecture */
 # patch @0x12: 0x3E -> 0x03
 
-if { ![istarget "i?86-*-*"] && ![istarget "x86_64-*-*"] } then {
+if {![istarget "i?86-*-*"] && ![istarget "x86_64-*-*"]} {
     verbose "Skipping i386 biarch core test."
     return
 }
index f45871c4b588cf282b918d91ed175e7d8f00ba62..975c3559af8070860dbe0e0c2ac0c991cff9a973 100644 (file)
@@ -18,7 +18,7 @@
 
 # Test stepping over permanent breakpoints on i386.
 
-if { ![istarget "i?86-*-*"] && ![istarget "x86_64-*-*"] } then {
+if {![istarget "i?86-*-*"] && ![istarget "x86_64-*-*"]} {
     verbose "Skipping skip over permanent breakpoint on i386 tests."
     return
 }
@@ -45,7 +45,7 @@ gdb_load ${binfile}
 # Run to `main' where we begin our tests.
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
   return -1
 }
 
index f29d54ec1bc5630170bcf52115c5d022b7685909..e6564f731629a9b2412ad33796cd7cd29808d306 100644 (file)
@@ -19,7 +19,7 @@
 # This file is part of the gdb testsuite.
 
 
-if {(![istarget "x86_64-*-*"] && ![istarget "i?86-*-*"])} then {
+if {(![istarget "x86_64-*-*"] && ![istarget "i?86-*-*"])} {
     verbose "Skipping i386 byte register tests."
     return
 }
@@ -33,7 +33,7 @@ if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable [list
 
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index c02b31f8b9a6e22312598b36f658099a2ec1e56f..dba61a67736cb1f8429d3e4160590a355858ff10 100644 (file)
@@ -18,7 +18,7 @@
 # Test i386 displaced stepping.
 
 
-if { ![is_x86_like_target] } then {
+if {![is_x86_like_target]} {
     verbose "Skipping x86 displaced stepping tests."
     return
 }
@@ -32,7 +32,7 @@ if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
 gdb_test "set displaced-stepping on" ""
 gdb_test "show displaced-stepping" ".* displaced stepping .* is on.*"
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index bab118b0ea0badbdc90bee11b4b51aec6cfef036..305cfea5e68af260df1896658fa3385111b6575a 100644 (file)
@@ -19,7 +19,7 @@
 # This test relies on being able to set 4 hardware watchpoints.  Since
 # that is not a valid assumption across most targets, and we're
 # testing a x86 specific bug, skip everywhere else.
-if { ![istarget "i?86-*-*"] && ![istarget "x86_64-*-*"] } then {
+if {![istarget "i?86-*-*"] && ![istarget "x86_64-*-*"]} {
     return 0
 }
 
@@ -29,7 +29,7 @@ if [prepare_for_testing "failed to prepare" ${binfile} ${srcfile} {debug $additi
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index 033c0eac900abc60abb95e778ace17ad9f6f1973..69f673534d48c87131f960d74e84bcd385978c49 100644 (file)
@@ -18,7 +18,7 @@
 
 # Test the x87 floating point information printout.
 
-if { ![istarget "i?86-*-*"] && ![istarget "x86_64-*-*"] } then {
+if {![istarget "i?86-*-*"] && ![istarget "x86_64-*-*"]} {
     verbose "Skipping i386 tests for x87 floating point support."
     return
 }
@@ -32,7 +32,7 @@ if { [prepare_for_testing "failed to prepare" $testfile $srcfile [list debug nop
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 68aae8f92360a8e0c764556e4a23dc44d1dc0567..04f80acf44a225c10fb7b8fd464d45428fdb3f03 100644 (file)
@@ -23,7 +23,7 @@
 # Test i386 unwinder.
 
 
-if { ![is_x86_like_target] } then {
+if {![is_x86_like_target]} {
     verbose "Skipping i386 unwinder tests."
     return
 }
@@ -86,7 +86,7 @@ gdb_test "frame $gate_n" \
 gdb_test_multiple "info frame" "existence of the CFI inserted register" {
     -re "Stack level $gate_n, frame at (0x\[0-9a-f\]+):.*Saved registers:.* ecx at (0x\[0-9a-f\]+),.*" {
        pass "existence of the CFI inserted register"
-       if { [string compare $expect_out(1,string) $expect_out(2,string)] } then {
+       if {[string compare $expect_out(1,string) $expect_out(2,string)]} {
            fail "value of the CFI inserted register"
        } else {
            pass "value of the CFI inserted register"
index 48b6b1574a5003dfd935dba83423337b6864b947..3e8c6b52accf840ecd621b744b72d11550e8bd0a 100644 (file)
@@ -18,7 +18,7 @@
 
 # Test inserting breakpoints over permanent breakpoints on i386 and amd64.
 
-if { ![istarget "i?86-*-*"] && ![istarget "x86_64-*-*"] } then {
+if {![istarget "i?86-*-*"] && ![istarget "x86_64-*-*"]} {
     verbose "Skipping i386 test for multi break at permanent breakpoint location."
     return
 }
index 9cfd7e5ccc57b013123cf1689925336f71a4260a..1988e0b7b85d2669525dee14d978d96388c6c3be 100644 (file)
@@ -21,7 +21,7 @@
 # Test i386 prologue analyzer.
 
 
-if { ![is_x86_like_target] } then {
+if {![is_x86_like_target]} {
     verbose "Skipping i386 prologue tests."
     return
 }
@@ -65,7 +65,7 @@ gdb_load ${binfile}
 # Run to `main' where we begin our tests.
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 7e8f62350a376f9fcfd4d59c352b464441f5133a..d7677c8bb01f10092538cb70bec97292e7d16379 100644 (file)
@@ -15,7 +15,7 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-if { ![istarget "i?86-*-*"] && ![istarget "x86_64-*-*"] } then {
+if {![istarget "i?86-*-*"] && ![istarget "x86_64-*-*"]} {
     verbose "Skipping i386 unwinder tests."
     return
 }
index e7ea47e4aca482e02dca7885ae4941867bd1caa2..ecdbb0a8e114f4169654c7c8698005004399ba5c 100644 (file)
@@ -17,7 +17,7 @@
 
 # Test that GDB can handle overlapping sizes of symbols.
 
-if { ![is_x86_like_target] } then {
+if {![is_x86_like_target]} {
     verbose "Skipping i386 unwinder tests."
     return
 }
index 4fdd3305c4bfc4f1d58b2407376a6946ae736f40..19c5552631a65e3cb5708dda4ed3ed35d2fe66ce 100644 (file)
@@ -20,7 +20,7 @@
 
 # Test that GDB can see the sizes of symbols.
 
-if { ![is_x86_like_target] } then {
+if {![is_x86_like_target]} {
     verbose "Skipping i386 unwinder tests."
     return
 }
index c66a352a9d10c4cb6c5a822edae3c2e2b316e487..e4699fe511b1ae57a55593a3ad86265c079c479d 100644 (file)
@@ -37,7 +37,7 @@ if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable $opts
 
 clean_restart $executable
 
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index 6565f15aa760d9fa5ca2a8e3e4a50a959892a46f..2f94fc4521d5d69e68bc067178f481ba5d72ddb1 100644 (file)
@@ -43,7 +43,7 @@ if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable \
 
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 66f30bcb8c4e1072e14222b6548751cab6ec6f8b..f9f0307674fc60267b8943c32415a5bc988deb4f 100644 (file)
@@ -21,7 +21,7 @@
 # Test i386 unwinder.
 
 
-if { ![is_x86_like_target] } then {
+if {![is_x86_like_target]} {
     verbose "Skipping i386 unwinder tests."
     return
 }
index 5ad01eb20f10be58d55cf7b4479d6ae3c7a46933..16d7dbfcc53090e065161f2e0f9ac928b1ba9368 100644 (file)
@@ -19,7 +19,7 @@
 # This file is part of the gdb testsuite.
 
 
-if {(![istarget "x86_64-*-*"] && ![istarget "i?86-*-*"])} then {
+if {(![istarget "x86_64-*-*"] && ![istarget "i?86-*-*"])} {
     verbose "Skipping i386 word register tests."
     return
 }
@@ -33,7 +33,7 @@ if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable [list
 
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 1e96937f6797ebf2256252aee9e7c3df75f8a192..1d8be6e706694adbbb6fb2245958b86b76d569e7 100644 (file)
@@ -16,7 +16,7 @@
 
 # Tests for ARM iWMMXt register setting and fetching.
 
-if ![istarget "arm*-*-*"] then {
+if {![istarget "arm*-*-*"]} {
     verbose "Skipping iWMMXt register tests."
     return
 }
@@ -40,7 +40,7 @@ gdb_load ${binfile}
 # Run to `main' where we begin our tests.
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index fd363b727de8fcd6ee4c87b6d4fbc772f26f0f60..97a2d5b4f663ab35d559fb2ad3a362eb35c179de 100644 (file)
@@ -15,7 +15,7 @@
 
 # Test MIPS disassembler options.
 
-if { ![istarget "mips*-*-*"] } then {
+if {![istarget "mips*-*-*"]} {
     verbose "Skipping MIPS disassembler option tests."
     return
 }
index 294e6f0c34d5080824e848da2f1d20023a810d3a..77b48bfe1810cc714718200bc794844b3bdf8eac 100644 (file)
@@ -17,7 +17,7 @@
 
 # Test MIPS Floating Point Control Register handling.
 
-if { ![istarget "mips*-*-*"] } then {
+if {![istarget "mips*-*-*"]} {
     verbose "Skipping MIPS Floating Point Control Register tests."
     return
 }
index 059c227aab8f4fc5b92f3ac6ebebf33d26c06e7f..bbfa1b5e6c11d4dc559695472ade4a90ac1b1955 100644 (file)
@@ -15,7 +15,7 @@
 
 # Test MIPS Floating Point General Register handling in core files.
 
-if { ![istarget "mips*-*-*"] } then {
+if {![istarget "mips*-*-*"]} {
     verbose "Skipping MIPS Floating Point General Register tests."
     return
 }
@@ -133,7 +133,7 @@ proc mips_fpregset_core_test { supported corefile } {
        set test "core file FGR register contents"
        set core_fgr_contents \
            [mips_fpregset_core_fetch_float_registers "retrieve $test"]
-       if { $core_fgr_contents == $live_fgr_contents } then {
+       if {$core_fgr_contents == $live_fgr_contents} {
            pass $test
        } else {
            fail $test
index 00ebeecb507ad738135b56449b9b0857d522e19e..d0b9038544a1c552ee09f673449070b0740da905 100644 (file)
@@ -25,7 +25,7 @@
 # toolchain for a subset of ABIs, so we need to check that a MIPS16
 # executable can be built and run at all before we attempt the actual test.
 
-if { ![istarget "mips*-*-*"] } then {
+if {![istarget "mips*-*-*"]} {
     verbose "Skipping MIPS16 thunk support tests."
     return
 }
@@ -37,7 +37,7 @@ proc set_src_and_obj { filename { suffix "" } } {
     global srcdir
     global subdir
 
-    if ![string equal "$suffix" ""] then {
+    if {![string equal "$suffix" ""]} {
        set suffix "-$suffix"
     }
     set srcfile ${srcdir}/${subdir}/${filename}.c
@@ -61,7 +61,7 @@ gdb_compile ${srcfile} ${objfile} object ${options}
 
 set binfile [standard_output_file ${testname}]
 set options [list debug nowarnings]
-if { [gdb_compile ${objfiles} ${binfile} executable ${options}] != "" } then {
+if {[gdb_compile ${objfiles} ${binfile} executable ${options}] != ""} {
     unsupported "no MIPS16 support in the toolchain."
     return
 }
@@ -116,7 +116,7 @@ gdb_compile ${srcfile} ${objfile} object ${options}
 
 set binfile [standard_output_file ${testname}]
 set options [list debug nowarnings additional_flags=-fPIC]
-if { [gdb_compile ${objfiles} ${binfile} executable ${options}] == "" } then {
+if {[gdb_compile ${objfiles} ${binfile} executable ${options}] == ""} {
     clean_restart ${testname}
     gdb_breakpoint inmain
     gdb_run_cmd
@@ -189,7 +189,7 @@ set binfile [standard_output_file ${testname}]
 set options [list debug nowarnings]
 gdb_compile ${objfiles} ${binfile} executable ${options}
 clean_restart ${testname}
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
@@ -291,7 +291,7 @@ proc finish_through { id funcs } {
 
 # Report PASS if VAL is equal to EXP, otherwise report FAIL, using MSG.
 proc pass_if_eq { val exp msg } {
-    if [string equal "$val" "$exp"] then {
+    if {[string equal "$val" "$exp"]} {
        pass "$msg"
     } else {
        fail "$msg"
@@ -304,7 +304,7 @@ proc pass_if_eq { val exp msg } {
 # for the same function.  If successful, then return the name of the
 # function we have stopped in.
 proc finish_if_ne { id func want funcs } {
-    if ![string equal "$func" "$want"] then {
+    if {![string equal "$func" "$want"]} {
        set call "$func"
        set want [lindex $funcs 0]
        set func [finish_through "$id" [linsert $funcs 0 "$func"]]
index bb00f57249bc5da6fb7e1616b7112f60ae07bd84..2c30f06224be5b6bd017a9c5baa575374631c061 100644 (file)
@@ -88,7 +88,7 @@ proc get_addr_of_sym { sym } {
   return $addr
 }
 
-if  ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
@@ -124,7 +124,7 @@ proc test_core_bt { test } {
 }
 
 set test "core at last insn in foo"
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 gdb_breakpoint "*$foo_last"
@@ -134,7 +134,7 @@ if [gen_core $test] {
 }
 
 set test "core at nullified insn"
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 gdb_breakpoint "*$foo_last"
index 12b2141cdb05c9dda92be99133315cf57f3c01ce..e7eecf4ffa5612bfe8cda9d09d3a0cc38c00368c 100644 (file)
@@ -38,7 +38,7 @@ if { [prepare_for_testing "failed to prepare" $testfile "$srcfile" \
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
       return
 }
 
index a85891bab9fa9d71736396f1caa73b4e2102a232..bd959bb056b015db4c1d8092656d1f00b0fb9b89 100644 (file)
@@ -18,7 +18,7 @@
 # Test rs6000 prologue analyzer.
 
 
-if ![istarget "powerpc-*-aix*"] then {
+if {![istarget "powerpc-*-aix*"]} {
     verbose "Skipping powerpc-aix prologue tests."
     return
 }
@@ -62,7 +62,7 @@ proc insert_breakpoint {function expected_location} {
     # instruction at that address (we're not interested in the insn
     # itself, but rather at the address printed at the begining of
     # the instruction).
-    if { $address != "" } then {
+    if {$address != ""} {
        gdb_test "x /i $address" \
                 ".*<$function\\+$expected_location>.*" \
                 "check $function breakpoint address"
index 93c6d8cd96012dc0dcbd91e4bdd09df6c019e599..20f4d07ce19e228215dcef313d1c7f22442afe91 100644 (file)
@@ -20,7 +20,7 @@
 standard_testfile .s
 set objfile [standard_output_file ${testfile}.o]
 
-if {![istarget "powerpc*-*-*"]} then {
+if {![istarget "powerpc*-*-*"]} {
     verbose "Skipping PowerPC instructions disassembly."
     return
 }
index 7efc4fb750dfe9d994e2c1ef34b0162c2f6c90b5..05edfbd857373b9e2f16a95791ef0b63aaa4fdbf 100644 (file)
@@ -20,7 +20,7 @@
 standard_testfile .s
 set objfile [standard_output_file ${testfile}.o]
 
-if {![istarget "powerpc*-*-*"]} then {
+if {![istarget "powerpc*-*-*"]} {
     verbose "Skipping PowerPC instructions disassembly."
     return
 }
index a2d481b0c2fa9fc31e23229f014b8c614ad41ed2..dc36c5fc45624bc1d570272274cfb62fea32e337 100644 (file)
@@ -20,7 +20,7 @@
 standard_testfile .s
 set objfile [standard_output_file ${testfile}.o]
 
-if {![istarget "powerpc*-*-*"]} then {
+if {![istarget "powerpc*-*-*"]} {
     verbose "Skipping PowerPC instructions disassembly."
     return
 }
index 6e7bb675c8d16148f03a0a73d33e335aae40ea93..5c2c556d113e1f6840ca7746236418eaeb6b8097 100644 (file)
@@ -20,7 +20,7 @@
 
 # Testcase for ppc decimal128 pseudo-registers.
 
-if ![istarget "powerpc64*-*"] then {
+if {![istarget "powerpc64*-*"]} {
     verbose "Skipping powerpc Decimal128 pseudo-registers testcase."
     return
 }
@@ -39,7 +39,7 @@ gdb_start
 gdb_reinitialize_dir $srcdir/$subdir
 gdb_load ${binfile}
 
-if { ![runto_main] } then {
+if {![runto_main]} {
    return
 }
 
index beea9cff2474623ca35ae6cd0f66660177915ef0..3c096b012b998e21971cb075ea71bd1bc7ee89c5 100644 (file)
@@ -18,7 +18,7 @@
 # This test exercises set/show disassembler-options results are preserved
 # across multiple set architecture calls.
 
-if {![istarget "powerpc*-*-*"]} then {
+if {![istarget "powerpc*-*-*"]} {
     verbose "Skipping PowerPC disassembler options."
     return
 }
index 4703bc65adbaa6a6b068f9deba2e0da3edcc57f7..30032ee5f8c05f3899eb018815d3f7bf493fc05c 100644 (file)
@@ -18,7 +18,7 @@
 # This tests checks that generating and loading a core file preserves
 # the correct FPSCR size.
 
-if {![istarget "powerpc*-*-linux*"]} then {
+if {![istarget "powerpc*-*-linux*"]} {
     verbose "Skipping PowerPC test for corefiles with FPSCR."
     return
 }
@@ -39,7 +39,7 @@ if {[build_executable "compile" $binfile $gen_src] == -1} {
 
 clean_restart $binfile
 
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index ff49e1dd83ae30e84fff637cae61a8b14d572371..236e88065e2619f84f34817479738ad69815c3e7 100644 (file)
@@ -25,7 +25,7 @@
 # registers are then checked against the saved values, because the
 # abort should have reverted the registers to these values.
 
-if {![istarget "powerpc*-*-linux*"]} then {
+if {![istarget "powerpc*-*-linux*"]} {
     verbose "Skipping PowerPC test for HTM registers."
     return
 }
index e680a531d5ff2e58fbe76a4742063a354590f901..f2beca1f15d618893168c011050bf70c633610d8 100644 (file)
@@ -37,7 +37,7 @@ if { [prepare_for_testing "failed to prepare" $testfile "$srcfile" \
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
       return
 }
 
index 8a79e810fbd6c38bfb3ac59ec4b503d7f1c69029..d5a1671a9e4cc2f3d6ecc762f55a0bc9c2ccb0a8 100644 (file)
@@ -33,7 +33,7 @@ if { [prepare_for_testing "failed to prepare" $testfile "$srcfile" \
 gdb_test "set radix 0b10000"
 gdb_test "set debug displaced"
 
-if ![runto_main] then {
+if {![runto_main]} {
       return
 }
 
index b9383d8bd2abc65b72efedf8c8dbb1390cbd36df..10cd3942bbf1b04071d2caf2c33a503da837b042 100644 (file)
@@ -20,7 +20,7 @@
 standard_testfile .s
 set objfile [standard_output_file ${testfile}.o]
 
-if {![istarget "powerpc*-*-*"]} then {
+if {![istarget "powerpc*-*-*"]} {
     verbose "Skipping PowerPC instructions disassembly."
     return
 }
index 10f7eebefc9bb5b1625ef4a38e3bf79c89b4c8af..3e4b51a89cfd4c343e57a31a3e6811d90b040ef9 100644 (file)
@@ -20,7 +20,7 @@
 standard_testfile .s
 set objfile [standard_output_file ${testfile}.o]
 
-if {![istarget "powerpc*-*-*"]} then {
+if {![istarget "powerpc*-*-*"]} {
     verbose "Skipping PowerPC instructions disassembly."
     return
 }
index cb0b3eb523309f8f04093ef1f9e35b1e9014d93c..7cb6ca1a126199b654a678cb77b887bf0bc396f5 100644 (file)
@@ -20,7 +20,7 @@
 standard_testfile .s
 set objfile [standard_output_file ${testfile}.o]
 
-if {![istarget "powerpc*-*-*"]} then {
+if {![istarget "powerpc*-*-*"]} {
     verbose "Skipping PowerPC instructions disassembly."
     return
 }
index b43cd3baf37e23b94f6deec7bc05a811085156ee..1a44e5520c9e302f7bc0cba598e7acb37b809521 100644 (file)
@@ -20,7 +20,7 @@
 standard_testfile .s
 set objfile [standard_output_file ${testfile}.o]
 
-if {![istarget "powerpc*-*-*"]} then {
+if {![istarget "powerpc*-*-*"]} {
     verbose "Skipping PowerPC instructions disassembly."
     return
 }
index a2d5a171f48d9ae255de1842b72fdd1c54f3bac7..5d4581b35502732982fad706472330bb45d2f0ef 100644 (file)
@@ -20,7 +20,7 @@
 # values, then write to the registers trough GDB, step once, and check
 # again if we read back the same values.
 
-if {![istarget "powerpc*-*-linux*"]} then {
+if {![istarget "powerpc*-*-linux*"]} {
     verbose "Skipping PowerPC test for PPR and DSCR registers."
     return
 }
index b31c046c5f7994441b41040f4edd407d9ea46dfe..1403fa318ea6e7bf29d2864785c0ffc7047241d6 100644 (file)
@@ -17,7 +17,7 @@
 
 # Do not run on AIX (where we won't be able to build the tests without
 # some surgery) or on PowerPC64 (ditto, dot symbols).
-if {[istarget *-*-aix*] || ![istarget "powerpc-*-*"]} then {
+if {[istarget *-*-aix*] || ![istarget "powerpc-*-*"]} {
     verbose "Skipping PowerPC prologue tests."
     return
 }
@@ -43,7 +43,7 @@ gdb_load ${binfile}
 # Run to `main' where we begin our tests.
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 2323929e9a3cfb1f74d174768c6694cd795ac044..4fcb76419fce8c79ec98ea989684f971a2f1afa3 100644 (file)
@@ -15,7 +15,7 @@
 
 # Testcase for PR tdep/17379.
 
-if {![istarget "powerpc*-*-*"]} then {
+if {![istarget "powerpc*-*-*"]} {
     verbose "Skipping powerpc-stackless.exp"
     return
 }
index 7b6d186050b2d3d58cae39973b5f4c8b243efd4d..6d82a2a23dfdeb232ecf91572c1a5154d4e5d66a 100644 (file)
@@ -22,7 +22,7 @@
 # branch to TAR and check that we stop at the address that we wrote to
 # register.
 
-if {![istarget "powerpc*-*-linux*"]} then {
+if {![istarget "powerpc*-*-linux*"]} {
     verbose "Skipping PowerPC test for the TAR register."
     return
 }
index b65db282d383e4ef6af9e373d7f02b39cc7be0ec..076a99afd8cb8d0b057a13558b3144fe6d3b89f1 100644 (file)
@@ -18,7 +18,7 @@
 # This test checks that generating and loading a core file preserves
 # the correct VSX register state.
 
-if {![istarget "powerpc*-*-linux*"] || [skip_vsx_tests]} then {
+if {![istarget "powerpc*-*-linux*"] || [skip_vsx_tests]} {
     verbose "Skipping PowerPC test for corefiles with VSX registers."
     return
 }
@@ -39,7 +39,7 @@ if {[build_executable "compile" $binfile $gen_src] == -1} {
 
 clean_restart $binfile
 
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index c4e13c154f058ba6ca8cd124a71ec1aa531097aa..2cb891b65d60eff9f5e05c54bf9762d3111059ff 100644 (file)
@@ -20,7 +20,7 @@
 standard_testfile .s
 set objfile [standard_output_file ${testfile}.o]
 
-if {![istarget "powerpc*-*-*"]} then {
+if {![istarget "powerpc*-*-*"]} {
     verbose "Skipping PowerPC instructions disassembly."
     return
 }
index 05e4e98aee10575c59c967b3a53b2b86637c9766..3181fda91d56503598aeff7647acf6552a0b70a0 100644 (file)
@@ -20,7 +20,7 @@
 standard_testfile .s
 set objfile [standard_output_file ${testfile}.o]
 
-if {![istarget "powerpc*-*-*"]} then {
+if {![istarget "powerpc*-*-*"]} {
     verbose "Skipping PowerPC instructions disassembly."
     return
 }
index ad9f5b76e686e0f58fab8c31444bcdb12b6d8335..27ad96f37d0075eae05f101e7f8c7ca2533f0aea 100644 (file)
@@ -20,7 +20,7 @@
 standard_testfile .s
 set objfile [standard_output_file ${testfile}.o]
 
-if {![istarget "powerpc*-*-*"]} then {
+if {![istarget "powerpc*-*-*"]} {
     verbose "Skipping PowerPC instructions disassembly."
     return
 }
index 30ae74ed54f6f48e8777e93d9f34b175d8171675..ea2339252bb076a9d4b0d67797759366f046e82e 100644 (file)
@@ -17,7 +17,7 @@
 
 # Do not run on AIX (where we won't be able to build the tests without
 # some surgery) or on PowerPC64 (ditto, dot symbols).
-if {[istarget *-*-aix*] || ![istarget "powerpc*-*-*"]} then {
+if {[istarget *-*-aix*] || ![istarget "powerpc*-*-*"]} {
     verbose "Skipping PowerPC prologue tests."
     return
 }
@@ -37,7 +37,7 @@ if { [prepare_for_testing "failed to prepare" $testfile "$srcfile" \
 }
 
 # Run to `main' where we begin our tests.
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index ecaf2499a0f623e90f03637aa156fd10da0fb005..29eb7a31cb2076d9e75686dfb709dc6178f75d8e 100644 (file)
@@ -16,7 +16,7 @@
 
 # Tests for Powerpc Decimal Floating Point registers setting and fetching
 
-if ![istarget "powerpc*"] then {
+if {![istarget "powerpc*"]} {
     verbose "Skipping powerpc decimal floating point register tests."
     return
 }
index bd22d7c489e137a67664a9d3403e0da9b8bab800..dd5b9803ed4ae8ed384e05f8aa1bba33f7f9bf83 100644 (file)
@@ -16,7 +16,7 @@
 
 # Tests for Powerpc floating point register setting and fetching
 
-if ![istarget "powerpc*"] then {
+if {![istarget "powerpc*"]} {
     verbose "Skipping powerpc floating point register tests."
     return
 }
index 44f23d39381a19f65ad640b25d58086c4d709eb9..76573fbbd51c3a55c65a9a817230b0e504c9adec 100644 (file)
@@ -13,7 +13,7 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-if ![istarget "powerpc*"] then {
+if {![istarget "powerpc*"]} {
     verbose "Skipping powerpc long-double floating point tests."
     return
 }
@@ -27,13 +27,13 @@ proc do_test { name {opts {}} } {
     set lines [gdb_compile "${srcdir}/${subdir}/${srcfile}" "$binfile.$name" executable $ccopts]
     # We get this warning even with the nowarnings option ...
     regsub -all "(^|\n)\[^\n\]*using \[^\n\]* extended precision long double" $lines "" lines
-    if { $lines != "" } then {
+    if {$lines != ""} {
         return
     }
 
     clean_restart ${binfile}.${name}
 
-    if ![runto_main] then {
+    if {![runto_main]} {
         return
     }
 
index 4fc9f9e24751109bd65d8a6c0b4337b1d6c67214..919b1a9a549353534dc1ba6610dbb48c39c06fb2 100644 (file)
@@ -36,7 +36,7 @@ if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile} {debug quie
 proc do_test { displaced } {
     global decimal hex
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return -1
     }
 
index 4d1da73d7dd16b660993a0afa40cd395d810ffc8..c22bbf7f887377806e6b2037f9da810771ac4804 100644 (file)
@@ -45,7 +45,7 @@ if { [prepare_for_testing "failed to prepare" ${testfile} \
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 0f7ecd08e279cb2f2dc9547d15ac63f1b2cffe23..4316505c4dc7c367caa4798581047c4d3e08d5ee 100644 (file)
@@ -40,7 +40,7 @@ proc do_test { displaced } {
     global decimal hex
     global gdb_prompt inferior_exited_re srcfile srcfile2
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return -1
     }
 
index e4bbea223739db8a00a32a9c7f3ebf76bc0afa7c..a68a3d1b7df59391f290af2a5dcf0556d49e5e28 100644 (file)
@@ -16,7 +16,7 @@
 # Test proper disassembling of ARM thumb instructions when reloading a symbol
 # file.
 
-if {![is_aarch32_target]} then {
+if {![is_aarch32_target]} {
     verbose "Skipping ARM tests."
     return
 }
index 6506f62fb1e1cb519110b0e12fc69768ae55df6f..511e327226d0de88028f97add02b4713e838497a 100644 (file)
@@ -27,7 +27,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
    return 0
 }
 
index bf319650db34fe5b20257dec711a3805e1360748..0bc5009fac36160d59714ae5c5808fc2c511e432 100644 (file)
@@ -32,7 +32,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
    return 0
 }
 
index 4c188b78b1fb02291bf9dc181dac42b9db70b5cf..8ee237a741169c4e5be5126870dd3030e04af09d 100644 (file)
@@ -24,7 +24,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
    return 0
 }
 
index 385e16b575fde4c2ad3cb30fb28316b0634255d5..6d750b5e3a989003327f8d9f9ffd5793e73cdbb5 100644 (file)
@@ -39,7 +39,7 @@ foreach_with_prefix {insn_size} {6 8} {
        return -1
     }
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return 0
     }
 
index 04466e7184ed8ec99146dedb8bd50cd30ee1a48c..f07c04c71a6c6ccf74f1d24a4b6c76d2c71ca72d 100644 (file)
@@ -28,7 +28,7 @@ if {[prepare_for_testing "failed to prepare" $testfile \
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index c90bebb3f49fec0b745a895ac9c2918af451ed9f..212affb1f6d25cf12b433c683cf11edd72ecc504 100644 (file)
@@ -27,7 +27,7 @@ if {[prepare_for_testing "failed to prepare" $testfile \
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     fail "can't run to main"
     return 0
 }
index e27808abeaf114a110d6dccdc6b21079987fafba..7ba9e51cc3c2de0db2fc39bdce4bb4990667617c 100644 (file)
@@ -18,7 +18,7 @@
 # This test exercises set/show disassembler-options results are preserved
 # across multiple set architecture calls.
 
-if {![istarget "s390*-*-*"]} then {
+if {![istarget "s390*-*-*"]} {
     verbose "Skipping S390 disassembler options."
     return
 }
index aeaf3235ad689a6c0ad785e577e0be75a1538b05..41354832775d7477439789b7763cd7f93ce17cf8 100644 (file)
@@ -13,7 +13,7 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-if {![istarget "s390*-*-*"]} then {
+if {![istarget "s390*-*-*"]} {
     verbose "Skipping s390 stackless test."
     return
 }
index 35690103e092b287b49bca06eec4b1ef54676f53..6bbccb6083316597a668381951a0ff622a704407 100644 (file)
@@ -35,7 +35,7 @@ if { [prepare_for_testing "failed to prepare" $testfile $srcfile \
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index c88abaa82c878c024aa7f88b616e837570cc8be1..124811ca7ea3d7f9e06035475de2940b2c3ec220 100644 (file)
@@ -21,7 +21,7 @@
 # PC must be extracted from the sigreturn register save area in the
 # stack.
 
-if ![istarget "sparc*-*-linux*"] then {
+if {![istarget "sparc*-*-linux*"]} {
     return 0
 }
 
index ba1bce884c60b6fbf1490205a8f5ad08f522effb..d09345649567d183cdb7b63aff99f605c562d2c7 100644 (file)
@@ -18,7 +18,7 @@
 # Basic tests of examining/assigning ADI version tags, and reporting 
 # precise mismatch.
 
-if ![istarget "sparc64*-*-linux*"] then {
+if {![istarget "sparc64*-*-linux*"]} {
     verbose "Skipping sparc64 ADI test."
     return 0
 }
@@ -30,7 +30,7 @@ if { [prepare_for_testing "failed to prepare" $testfile $srcfile \
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index a617254863a48cf147502a9fb0fc0f1c2616bac8..3c786db7c60f226ca9b56554ef6d3e14a85ab993 100644 (file)
@@ -18,7 +18,7 @@
 # Tests decoding of various sparc64 registers.
 # At the moment, only few registers are tested, but more can be added in future.
 
-if ![istarget "sparc64*-*-linux*"] then {
+if {![istarget "sparc64*-*-linux*"]} {
     verbose "Skipping sparc64 register tests."
     return 0
 }
@@ -29,7 +29,7 @@ if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile}] } {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index ac62efa830c7ae5c6060839cb3be47c51b036537..ccc6fbfdca7d4b356cf307f5d49e4fa740d43d7b 100644 (file)
@@ -15,7 +15,7 @@
 
 # Test PC adjustment from Thumb-mode "bx pc" instruction.
 
-if {![is_aarch32_target]} then {
+if {![is_aarch32_target]} {
     verbose "Skipping ARM tests."
     return
 }
@@ -28,7 +28,7 @@ if [prepare_for_testing "failed to prepare" ${testfile} ${srcfile} $opts] {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index 0d3e6b01bd786634ea2c032cd7e40950f0cfb04b..0cb14572345808d35569e17ae49e98da9b642a0a 100644 (file)
@@ -15,7 +15,7 @@
 
 # Test ARM/Thumb prologue analyzer.
 
-if {![is_aarch32_target]} then {
+if {![is_aarch32_target]} {
     verbose "Skipping ARM prologue tests."
     return
 }
@@ -39,7 +39,7 @@ gdb_load ${binfile}
 # Run to `main' where we begin our tests.
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index f31e428b421127cfc5fb105ff7c01c737ad1cf3b..289bdf52463dc3aff5264984136b08f503630c37 100644 (file)
@@ -15,7 +15,7 @@
 
 # Test single-stepping into incorrectly marked Thumb routine
 
-if {![is_aarch32_target]} then {
+if {![is_aarch32_target]} {
     verbose "Skipping ARM tests."
     return
 }
@@ -28,7 +28,7 @@ if [prepare_for_testing "failed to prepare" ${testfile} ${srcfile} [list debug $
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index 3892c90d8eac91c4a8050310998f27b0e4cccdbd..0e1a2c2ea4e6cd940c1a635f6cbf2a07d76980c3 100644 (file)
@@ -15,7 +15,7 @@
 
 # Test single stepping over Thumb-2 IT blocks.
 
-if {![istarget arm*-*eabi*]} then {
+if {![istarget arm*-*eabi*]} {
     verbose "Skipping Thumb-2 tests."
     return
 }
@@ -32,7 +32,7 @@ gdb_start
 gdb_reinitialize_dir $srcdir/$subdir
 gdb_load ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index c288b2f7d6f2b7ccf259a05533460b132bb8aa89..67890154f6a2beba98710057d799d1992d43435f 100644 (file)
@@ -19,7 +19,7 @@
 #
 
 
-if {![istarget "powerpc*"] || [skip_vsx_tests]} then {
+if {![istarget "powerpc*"] || [skip_vsx_tests]} {
     verbose "Skipping vsx register tests."
     return
 }
@@ -47,7 +47,7 @@ gdb_load ${binfile}
 
 # Run to `main' where we begin our tests.
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index b5b2d5844031a88325e83ed41f6becf14b0de9a1..c2046977771164ed599b2226abd5f105073bc5ba 100644 (file)
@@ -19,7 +19,7 @@
 #
 
 
-if {![istarget "powerpc*"] || [skip_vsx_tests]} then {
+if {![istarget "powerpc*"] || [skip_vsx_tests]} {
     verbose "Skipping vsr float128 field tests."
     return
 }
@@ -47,7 +47,7 @@ gdb_load ${binfile}
 
 # Run to `main' where we begin our tests.
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }