gc.exp: Remove extraneous LD flags from the personality test
authorMaciej W. Rozycki <macro@mips.com>
Tue, 20 Mar 2018 11:48:59 +0000 (11:48 +0000)
committerMaciej W. Rozycki <macro@mips.com>
Tue, 20 Mar 2018 11:48:59 +0000 (11:48 +0000)
Complement commit 8988502d7534 ("MIPS/LD/testsuite: Correct dynamic
links with VR4100, VR4300 and VR5000") and commit bf48520113c4 ("gc.exp:
Fix a typo: $LFLAGS -> $LDFLAGS"), and remove extraneous additional LD
flags explicitly passed to the personality test on invocation.  This
passing has accidentally leaked from a WIP version of the former change.

With the actual version committed the $LDFLAGS global variable is used
instead for `run_dump_test' to use implicitly, so that `-call_shared'
precedes `-lpersonality' on the linker's command line, as otherwise a
shared `libpersonality.so' library wouldn't be considered for linking,
causing the test to fail.

ld/
* testsuite/ld-gc/gc.exp: Remove extraneous LD flags from the
personality test.

ld/ChangeLog
ld/testsuite/ld-gc/gc.exp

index 2eaf710..7270ac0 100644 (file)
@@ -1,3 +1,8 @@
+2018-03-20  Maciej W. Rozycki  <macro@mips.com>
+
+       * testsuite/ld-gc/gc.exp: Remove extraneous LD flags from the
+       personality test.
+
 2018-03-20  H.J. Lu  <hongjiu.lu@intel.com>
 
        PR ld/22983
index efdddd7..75d5714 100644 (file)
@@ -127,7 +127,7 @@ if { [is_elf_format] && [check_shared_lib_support] } then {
        || ![ld_link $ld tmpdir/libpersonality.so "-shared tmpdir/libpersonality.o"] } then {
        fail libpersonality
     } else {
-       run_dump_test "personality" [list [list ld $LDFLAGS]]
+       run_dump_test "personality"
     }
     run_dump_test "pr18223"
     if {![ld_assemble_flags $as $gasopt $srcdir/$subdir/pr20022a.s tmpdir/pr20022a.o]