From: Alan Modra Date: Wed, 1 Aug 2018 09:29:03 +0000 (+0930) Subject: Re: PowerPC64 __tls_get_addr_opt stub .eh_frame fix X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=a67ae9b01af7eb2b1f8e5c517338a4b12167ce40;p=binutils-gdb.git Re: PowerPC64 __tls_get_addr_opt stub .eh_frame fix The modified test failed on some powerpc targets due to differences in default hash style. If the default hash style is both, then more sections are created, bumping section ids. Section id is used in stub symbols and although the test is careful to not depend on id in labels, the stub hash traversal order changes when stub names change. That lead to the stubs being emitted in a different order and thus not matching expected output. * testsuite/ld-powerpc/powerpc.exp: Run tlsopt5 with hash-style specified. --- diff --git a/ld/ChangeLog b/ld/ChangeLog index 73e57b30d06..93b3f6729ee 100644 --- a/ld/ChangeLog +++ b/ld/ChangeLog @@ -1,3 +1,8 @@ +2018-08-01 Alan Modra + + * testsuite/ld-powerpc/powerpc.exp: Run tlsopt5 with hash-style + specified. + 2018-08-01 Alan Modra * testsuite/ld-powerpc/powerpc.exp: Run tlsopt5 with plt alignment. diff --git a/ld/testsuite/ld-powerpc/powerpc.exp b/ld/testsuite/ld-powerpc/powerpc.exp index ed7bc74cf42..ac563eba142 100644 --- a/ld/testsuite/ld-powerpc/powerpc.exp +++ b/ld/testsuite/ld-powerpc/powerpc.exp @@ -224,7 +224,7 @@ set ppc64elftests { "tlsopt4"} {"TLS DLL" "-shared -melf64ppc --version-script tlsdll.ver" "" "-a64" {tlsdll.s} {} "tlsdll.so"} - {"TLS opt 5" "-melf64ppc -shared --gc-sections --no-plt-localentry tmpdir/tlsdll.so" "" "-a64" {tlsopt5.s} + {"TLS opt 5" "-melf64ppc -shared --hash-style=both --gc-sections --no-plt-localentry tmpdir/tlsdll.so" "" "-a64" {tlsopt5.s} {{objdump -dr tlsopt5.d} {readelf -wf tlsopt5.wf}} "tlsopt5"} {"sym@tocbase" "-shared -melf64ppc" "" "-a64" {symtocbase-1.s symtocbase-2.s}