From 6bd454ca033dcb2e178721eb004d0c5f24415c4c Mon Sep 17 00:00:00 2001 From: Alan Modra Date: Wed, 30 Nov 2022 17:09:20 +1030 Subject: [PATCH] Correct ordering problem in comm-data.exp * testsuite/ld-elf/comm-data.exp: Build libcomm-data.so before attempting to read it to set ELF64. --- ld/testsuite/ld-elf/comm-data.exp | 39 +++++++++++++++---------------- 1 file changed, 19 insertions(+), 20 deletions(-) diff --git a/ld/testsuite/ld-elf/comm-data.exp b/ld/testsuite/ld-elf/comm-data.exp index cea03df2366..f46d69350b6 100644 --- a/ld/testsuite/ld-elf/comm-data.exp +++ b/ld/testsuite/ld-elf/comm-data.exp @@ -48,19 +48,8 @@ if { [istarget "mips*vr4100*-*-elf*"] \ append LFLAGS " -call_shared" } -# Set the pointer size according to the ELF flavor. -set AFLAGS "" -if [is_elf64 "tmpdir/libcomm-data.so"] { - append AFLAGS " --defsym ELF64=1" -} -# HPUX targets use a different .comm syntax. -if [istarget "*-*-hpux*"] { - append AFLAGS " --defsym HPUX=1" -} - -set testname "Common symbol override test" - # Define a global symbol. +set testname "Common symbol override test" run_ld_link_tests [list \ [list \ "$testname (auxiliary shared object build)" \ @@ -72,16 +61,18 @@ run_ld_link_tests [list \ } \ "libcomm-data.so" \ ] \ - [list \ - "libpr26580-1.so" \ - "$LFLAGS -shared" "" \ - "$AFLAGS_PIC $AFLAGS" \ - { pr26580-b.s } \ - { } \ - "libpr26580-1.so" \ - ] \ ] +# Set the pointer size according to the ELF flavor. +set AFLAGS "" +if [is_elf64 "tmpdir/libcomm-data.so"] { + append AFLAGS " --defsym ELF64=1" +} +# HPUX targets use a different .comm syntax. +if [istarget "*-*-hpux*"] { + append AFLAGS " --defsym HPUX=1" +} + # bfin-elf does not currently support copy relocs. setup_xfail "bfin-*-*" clear_xfail "bfin-*-linux-uclibc*" @@ -121,6 +112,14 @@ run_ld_link_tests [list \ } \ "comm-data3b" \ ] \ + [list \ + "libpr26580-1.so" \ + "$LFLAGS -shared" "" \ + "$AFLAGS_PIC $AFLAGS" \ + { pr26580-b.s } \ + { } \ + "libpr26580-1.so" \ + ] \ [list \ "pr26580-1" \ "$LFLAGS --as-needed -Ltmpdir -lpr26580-1" "" \