gas: avoid spurious failures in non-ELF targets in the SPARC testsuite.

[deliverable/binutils-gdb.git] / gas / testsuite / gas / sparc / sparc.exp

diff --git a/gas/testsuite/gas/sparc/sparc.exp b/gas/testsuite/gas/sparc/sparc.exp
index f0a129da3bcbcfac8f9540485fe1185765308c68..8cb09046df92044d77b84a30be2d8097ba744656 100644 (file)
--- a/gas/testsuite/gas/sparc/sparc.exp
+++ b/gas/testsuite/gas/sparc/sparc.exp
@@ -31,25 +31,47 @@ proc gas_64_check { } {
     return [regexp "elf64\[_-\]sparc" [lindex $status 1]]
 }
 
-proc sparc_elf_setup { } {
-    setup_xfail "sparc*-*-*aout*" "sparc*-*-sunos4*"
-    setup_xfail "sparc*-fujitsu-none" "sparc*-*-*n*bsd*"
-    setup_xfail "sparc*-*-coff" "sparc*-*-lynxos*"
-    clear_xfail "sparc64*-*-*n*bsd*"
-    clear_xfail "sparc*-*-netbsdelf*"
-}
-
 if [istarget sparc*-*-*] {
-    run_dump_test "synth"
-    # The next four tests are ELF only.
-    sparc_elf_setup
-    run_dump_test "unalign"
-    sparc_elf_setup
-    run_dump_test "pcrel"
-    sparc_elf_setup
-    run_dump_test "plt"
-    sparc_elf_setup
-    run_dump_test "gotop32"
+
+    if [is_elf_format] {
+        # The next five tests are ELF only.
+        run_dump_test "unalign"
+        run_dump_test "pcrel"
+        run_dump_test "plt"
+        run_dump_test "gotop32"
+        run_dump_test "pc2210"
+        # The features tested in the tests below are not ELF-specific.
+        # However, these features are not present in practice in COFF
+        # and a.out targets.  Running them only in ELF targets allows
+        # us to don't have to replicate pattern test files as that
+        # would bring no practical advantage.
+        run_dump_test "natural"
+        run_dump_test "natural-32"
+        run_list_test "pr4587" ""
+        run_dump_test "ticc-imm-reg"
+        run_dump_test "v8-movwr-imm"
+        run_dump_test "pause"
+        run_dump_test "save-args"
+        run_dump_test "cbcond"
+        run_dump_test "cfr"
+        run_dump_test "crypto"
+        run_dump_test "edge"
+        run_dump_test "flush"
+        run_dump_test "hpcvis3"
+        run_dump_test "ima"
+        run_dump_test "ld_st_fsr"
+        run_dump_test "ldtw_sttw"
+        run_dump_test "ldd_std"
+        run_dump_test "ldx_stx"
+        run_dump_test "ldx_efsr"
+        run_dump_test "mwait"
+        run_dump_test "mcdper"
+        run_dump_test "sparc5vis4"
+        run_dump_test "xcrypto"
+        run_dump_test "v9branch1"
+        run_dump_test "imm-plus-rreg"
+    }
+
     if [gas_64_check] {
        run_dump_test "asi"
        run_dump_test "membar"
@@ -68,39 +90,12 @@ if [istarget sparc*-*-*] {
        run_dump_test "plt64"
        run_dump_test "gotop64"
     }
-    run_dump_test "imm-plus-rreg"
-    run_dump_test "ticc-imm-reg"
-    run_dump_test "v8-movwr-imm"
-    run_dump_test "save-args"
-    run_dump_test "v9branch1"
+
+    run_dump_test "synth"
     run_dump_test "v9branch2"
     run_dump_test "v9branch3"
     run_dump_test "v9branch4"
     run_dump_test "v9branch5"
-    run_dump_test "pc2210"
-    run_dump_test "hpcvis3"
-    run_dump_test "ima"
-    run_dump_test "crypto"
-    run_dump_test "xcrypto"
-    run_dump_test "cbcond"
-    run_dump_test "pause"
-    run_dump_test "cfr"
-    run_dump_test "ldtw_sttw"
-    run_dump_test "ldtxa"
-    run_dump_test "ldd_std"
-    run_dump_test "ldx_stx"
-    run_dump_test "ldx_efsr"
-    run_dump_test "ld_st_fsr"
-    run_dump_test "edge"
-    run_dump_test "flush"
-    run_dump_test "mwait"
-    run_dump_test "mcdper"
-    run_dump_test "sparc5vis4"
-    run_dump_test "natural"
-    run_dump_test "natural-32"
-
-    run_list_test "pr4587" ""
-
     run_dump_test "pr19910-1"
     run_list_test "pr19910-2"
 }