]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
ia64 assembler warning breaks ld tests
authorAlan Modra <amodra@gmail.com>
Thu, 31 Jul 2025 21:40:40 +0000 (07:10 +0930)
committerAlan Modra <amodra@gmail.com>
Fri, 1 Aug 2025 01:16:27 +0000 (10:46 +0930)
The "Warning: Explicit stops are ignored in auto mode" results in
failures of a number of run_ld_link_tests because the compiler is run
using -S and then the resulting .s file assembled without suppplying
-x to gas.  Fix that problem by adding -x to ASFLAGS for ia64, and
tweak the binutils link-order test since the source is used in a ld
test too.

ld/
* testsuite/config/default.exp: Set ASFLAGS to "-x" for ia64.
Remove unnecessary "global".
binutils/
* testsuite/binutils-all/link-order.s: Provide explicit stop.
* testsuite/binutils-all/objcopy.exp: Pass "-x" when building
link-order test for ia64.

binutils/testsuite/binutils-all/link-order.s
binutils/testsuite/binutils-all/objcopy.exp
ld/testsuite/config/default.exp

index 5994cb813a04ae8c71b78a7842e9e3b447880214..918cd06c0f20f1ac13cd89fe623483a7ba3256ba 100644 (file)
@@ -5,7 +5,7 @@
 foo:
        .prologue 2, 2
        .vframe r2
-       mov r2 = r12
+       mov r2 = r12;;
        .body
        .restore sp
        mov r12 = r2
index 6aa6d2d9fe7c29aa8586781bb90a4ce49e1ac758..89370bc893aedf17b24096748d292ae175ff214e 100644 (file)
@@ -1293,7 +1293,7 @@ proc objcopy_test_elf_common_symbols {} {
 # ia64 specific tests
 if { ([istarget "ia64-*-elf*"]
        || [istarget "ia64-*-linux*"]) } {
-    objcopy_test "ia64 link order" link-order.s object "" ""
+    objcopy_test "ia64 link order" link-order.s object "-x" ""
 }
 
 # ELF specific tests
index 1d696811e8e95e06e7ba50b81d53d948b89a3791..1c12ce601936a7c442bc04546594f5af63523c57 100644 (file)
@@ -185,10 +185,13 @@ if {[istarget mips64*-*-linux*] &&
 }
 
 if { [istarget rx-*-*] } {
-    global ASFLAGS
     set ASFLAGS "-muse-conventional-section-names"
 }
 
+if { [istarget ia64-*-*] } {
+    set ASFLAGS "-x"
+}
+
 # Blackfin ELF targets require selection of an explicit CPU.  Use the sim.
 if {[istarget bfin*-elf*]} {
     append gcc_B_opt " -msim"