A few hppa testcase tidies

binutils/
	* testsuite/lib/binutils-common.exp (is_som_format): New proc.
	(run_dump_test): Correct target test for alternate .comm syntax.
	(get_standard_section_names): Handle som format.
	* testsuite/lib/utils-lib.exp (default_binutils_assemble_flags):
	Correct target test for alternate .comm syntax.
gas/
	* testsuite/gas/all/gas.exp (redef3): Don't xfail for hppa.
	(octa): Run for hppa.
	* testsuite/gas/elf/elf.exp (common1, common2): Likewise.
	* testsuite/gas/elf/symver.d: Delete notarget.
ld/
	* testsuite/ld-elf/comm-data5.d: Remove notarget for hppa.
	* testsuite/ld-scripts/defined6.d: Likewise.

diff --git a/binutils/ChangeLog b/binutils/ChangeLog
index 0de86868f1e786bc10d323492d99be4e2b8db6c4..ee6defcbf8dd3a1e796d840df14c108d422e0c47 100644 (file)
--- a/binutils/ChangeLog
+++ b/binutils/ChangeLog
@@ -1,3 +1,11 @@
+2018-09-17  Alan Modra  <amodra@gmail.com>
+
+       * testsuite/lib/binutils-common.exp (is_som_format): New proc.
+       (run_dump_test): Correct target test for alternate .comm syntax.
+       (get_standard_section_names): Handle som format.
+       * testsuite/lib/utils-lib.exp (default_binutils_assemble_flags):
+       Correct target test for alternate .comm syntax.
+
 2018-09-17  Alan Modra  <amodra@gmail.com>
 
        * testsuite/binutils-all/strip-13mips64.s: Add section flags

diff --git a/binutils/testsuite/lib/binutils-common.exp b/binutils/testsuite/lib/binutils-common.exp
index 4aa5c7c97913a29d16e542757ecfc042c056a05e..08d57ae9638d6670077cc4fda347e79753fffc1c 100644 (file)
--- a/binutils/testsuite/lib/binutils-common.exp
+++ b/binutils/testsuite/lib/binutils-common.exp
@@ -105,6 +105,19 @@ proc is_pecoff_format {} {
     return 1
 }
 
+proc is_som_format {} {
+    if { ![istarget hppa*-*-*] || [istarget hppa*64*-*-*] } {
+       return 0;
+    }
+    if { [istarget *-*-osf*] \
+            || [istarget {*-*-h[ip]ux*}] \
+            || [istarget *-*-mpeix*] \
+            || [istarget *-*-bsd*] } {
+       return 1;
+    }
+    return 0;
+}
+
 # True if the object format is known to be 64-bit ELF.
 #
 proc is_elf64 { binary_file } {
@@ -945,7 +958,9 @@ proc run_dump_test { name {extra_options {}} } {
                set cmdret 1
            }
        } else {
-           if { [istarget "hppa*-*-*"] && ![istarget "*-*-linux*" ] } {
+           if { [istarget "hppa*-*-*"] \
+                    && ![istarget "*-*-linux*"] \
+                    && ![istarget "*-*-netbsd*" ] } {
                set cmd "sed -e 's/^\[   \]*\.comm \\(\[^,\]*\\),\\(.*\\)/\\1 .comm \\2/' < $sourcefile > tmpdir/asm.s"
                send_log "$cmd\n"
                set cmdret [remote_exec host [concat sh -c [list "$cmd"]]]
@@ -1328,9 +1343,8 @@ proc get_standard_section_names {} {
     if [istarget "rx-*-*"] {
        return { "P" "D_1" "B_1" }
     }
-    if [istarget "alpha*-*-*vms*"] {
-       # Double quote: for TCL and for sh.
-       return { "\\\$CODE\\\$" "\\\$DATA\\\$" "\\\$BSS\\\$" }
+    if { [istarget "alpha*-*-*vms*"] || [is_som_format] } {
+       return { {\$CODE\$} {\$DATA\$} {\$BSS\$} }
     }
     return
 }

diff --git a/binutils/testsuite/lib/utils-lib.exp b/binutils/testsuite/lib/utils-lib.exp
index 45c1f03c5e01f4ba411f6fa614b18fd881ed98fa..ace30e642732fdfa5c5cfcc8614645271f133cb1 100644 (file)
--- a/binutils/testsuite/lib/utils-lib.exp
+++ b/binutils/testsuite/lib/utils-lib.exp
@@ -117,7 +117,9 @@ proc default_binutils_assemble_flags { source object asflags } {
     # This is a hack in that it won't scale well if other targets need
     # similar transformations to assemble.  We'll generalize the hack
     # if/when other targets need similar handling.
-    if { [istarget "hppa*-*-*"] && ![istarget "*-*-linux*" ] } then {
+    if { [istarget "hppa*-*-*"] \
+            && ![istarget "*-*-linux*"] \
+            && ![istarget "*-*-netbsd*" ] } {
         set sed_file $srcdir/config/hppa.sed
        send_log "sed -f $sed_file < $source > asm.s\n"
        verbose "sed -f $sed_file < $source > asm.s"

diff --git a/gas/ChangeLog b/gas/ChangeLog
index 0fbe6fac0b9b381c8f23254e4e0b0ec44dc2f059..3772875cecce8e0c4a757979772e71656bf0a2b4 100644 (file)
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,10 @@
+2018-09-17  Alan Modra  <amodra@gmail.com>
+
+       * testsuite/gas/all/gas.exp (redef3): Don't xfail for hppa.
+       (octa): Run for hppa.
+       * testsuite/gas/elf/elf.exp (common1, common2): Likewise.
+       * testsuite/gas/elf/symver.d: Delete notarget.
+
 2018-09-17  Nick Clifton  <nickc@redhat.com>
 
        * testuite/gas/elf/group0a.d: Add extra details to the test

diff --git a/gas/testsuite/gas/all/gas.exp b/gas/testsuite/gas/all/gas.exp
index 029795beb8be09fb504fd2319892ca011780450a..b658da33e97786aeb7647845ae6cb3b741aaba7c 100644 (file)
--- a/gas/testsuite/gas/all/gas.exp
+++ b/gas/testsuite/gas/all/gas.exp
@@ -170,8 +170,6 @@ case $target_triplet in {
        setup_xfail "riscv*-*-*" "rx-*-*" "vax*-*-*" "xgate*-*-*" "z8k-*-*"
        # rs6000-aix disallows redefinition via .comm.
        setup_xfail "*-*-aix*"
-       # SOM uses a different syntax for .comm
-       setup_xfail "hppa*-*-hpux*"
        # These targets fail redef3 because section contents for the
        # word referencing the .comm sym is not zero and/or its reloc
        # has a non-zero addend.  Relaxing the test would hide real
@@ -414,17 +412,7 @@ if { ![istarget "powerpc*-*-*"] && ![istarget "rs6000*-*-*"] && ![istarget "s390
 }
 
 run_dump_test quad
-
-# som doesn't use .data section.
-case $target_triplet in {
-    { hppa*64*-*-* } {
-       run_dump_test octa
-    }
-    { hppa*-*-hpux* } { }
-    default {
-       run_dump_test octa
-    }
-}
+run_dump_test octa
 
 # .set works differently on some targets.
 case $target_triplet in {

diff --git a/gas/testsuite/gas/elf/elf.exp b/gas/testsuite/gas/elf/elf.exp
index 44f26c345d309a7ebf9a54a1ee001fad7c9596d9..9715ac837b0815864854ff8758b2ad62695a067f 100644 (file)
--- a/gas/testsuite/gas/elf/elf.exp
+++ b/gas/testsuite/gas/elf/elf.exp
@@ -252,10 +252,11 @@ if { [is_elf_format] } then {
 
     run_dump_test "missing-build-notes"
     
+    run_dump_test "common1"
+    run_dump_test "common2"
+
     # hpux has a non-standard common directive.
     if { ![istarget "*-*-hpux*"] } then {
-       run_dump_test "common1"
-       run_dump_test "common2"
        run_dump_test "common3a"
        run_dump_test "common3b"
        run_dump_test "common4a"

diff --git a/gas/testsuite/gas/elf/symver.d b/gas/testsuite/gas/elf/symver.d
index 3d180c73b03b108b61ecbea5595d444e8e19bbe9..7fcc62a5caa8a64d783ef11155c061ffae6c1dbd 100644 (file)
--- a/gas/testsuite/gas/elf/symver.d
+++ b/gas/testsuite/gas/elf/symver.d
@@ -1,6 +1,5 @@
 #objdump: --syms
 #name: ELF symbol versioning
-#notarget: hppa64*-*-hpux*
 #
 # The #... and #pass are there to match extra symbols inserted by
 # some toolchains, eg the mips-elf port will add .reginfo and .ptrd

diff --git a/ld/ChangeLog b/ld/ChangeLog
index 27e4c1becb0b1d9a19df8ced7cf63e59c581e351..d0e75595f8e68b797d85f2637faeea8dc865b38f 100644 (file)
--- a/ld/ChangeLog
+++ b/ld/ChangeLog
@@ -1,3 +1,8 @@
+2018-09-17  Alan Modra  <amodra@gmail.com>
+
+       * testsuite/ld-elf/comm-data5.d: Remove notarget for hppa.
+       * testsuite/ld-scripts/defined6.d: Likewise.
+
 2018-09-17  Nick Clifton  <nickc@redhat.com>
 
        * testuite/ld/ld-elf/audit.exp: Differentiate the names of the

diff --git a/ld/testsuite/ld-elf/comm-data5.d b/ld/testsuite/ld-elf/comm-data5.d
index 36c09ba750c0ba36d5e06f5783508fe899e2b4e0..18b287d122ba2400032f428b8ab50ea304d2263d 100644 (file)
--- a/ld/testsuite/ld-elf/comm-data5.d
+++ b/ld/testsuite/ld-elf/comm-data5.d
@@ -1,7 +1,6 @@
 #source: comm-data5.s
 #ld: 
 #readelf: -r
-#notarget: hppa64*-*-hpux*
 #xfail: frv-*-*linux*
 
 There are no relocations in this file.

diff --git a/ld/testsuite/ld-scripts/defined6.d b/ld/testsuite/ld-scripts/defined6.d
index c8ea86305893987a54a66b27b3ea429e0b092c20..8837f3b13edbd9f1b559ad5593e73834b18c4f10 100644 (file)
--- a/ld/testsuite/ld-scripts/defined6.d
+++ b/ld/testsuite/ld-scripts/defined6.d
@@ -1,8 +1,6 @@
 #source: defined6.s
 #ld: -T defined6.t
 #nm: -B
-#notarget: hppa*-hpux*
-# hppa64-hpux has a different .comm
 
 # Check that DEFINED works with various symbol types
 #...