From: H.J. Lu Date: Fri, 18 May 2018 21:23:41 +0000 (-0700) Subject: x86: Don't set eh->local_ref to 1 for versioned symbol X-Git-Tag: binutils-2_31~424 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=97373b2eba6077d5059370a95931b93a8b118813;p=external%2Fbinutils.git x86: Don't set eh->local_ref to 1 for versioned symbol bfd_hide_sym_by_version can't be used to check if a versioned symbol is hidden. It has to be synced with _bfd_elf_link_assign_sym_version to get the correct answer. bfd/ PR ld/23194 * elfxx-x86.c (_bfd_x86_elf_link_symbol_references_local): Don't set eh->local_ref to 1 if a symbol is versioned and there is a version script. ld/ PR ld/23194 * testsuite/ld-i386/i386.exp: Run pr23194. * testsuite/ld-x86-64/x86-64.exp: Likewise. * testsuite/ld-i386/pr23194.d: New file. * testsuite/ld-i386/pr23194.map: Likewise. * testsuite/ld-i386/pr23194.s: Likewise. * testsuite/ld-x86-64/pr23194.d: Likewise. * testsuite/ld-x86-64/pr23194.map: Likewise. * testsuite/ld-x86-64/pr23194.s: Likewise. --- diff --git a/bfd/ChangeLog b/bfd/ChangeLog index cc8773c..bab543b 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,10 @@ +2018-05-18 H.J. Lu + + PR ld/23194 + * elfxx-x86.c (_bfd_x86_elf_link_symbol_references_local): Don't + set eh->local_ref to 1 if a symbol is versioned and there is a + version script. + 2018-05-18 Kito Cheng * elfnn-riscv.c (_bfd_riscv_elf_merge_private_bfd_data): Handle diff --git a/bfd/elfxx-x86.c b/bfd/elfxx-x86.c index f4dbddf..29d92d2 100644 --- a/bfd/elfxx-x86.c +++ b/bfd/elfxx-x86.c @@ -2048,8 +2048,11 @@ _bfd_x86_elf_link_symbol_references_local (struct bfd_link_info *info, } /* Symbols created by HIDDEN and PROVIDE_HIDDEN assignments in linker - script aren't forced local here yet. */ - if (!h->root.ldscript_def) + script aren't forced local here yet. bfd_hide_sym_by_version + can't be used to check if a versioned symbol is hidden. It has to + be syncd with _bfd_elf_link_assign_sym_version to get the correct + answer. */ + if (!h->root.ldscript_def && h->versioned == unversioned) eh->local_ref = 1; return FALSE; diff --git a/ld/ChangeLog b/ld/ChangeLog index eb7fac4..15de953 100644 --- a/ld/ChangeLog +++ b/ld/ChangeLog @@ -1,5 +1,17 @@ 2018-05-18 H.J. Lu + PR ld/23194 + * testsuite/ld-i386/i386.exp: Run pr23194. + * testsuite/ld-x86-64/x86-64.exp: Likewise. + * testsuite/ld-i386/pr23194.d: New file. + * testsuite/ld-i386/pr23194.map: Likewise. + * testsuite/ld-i386/pr23194.s: Likewise. + * testsuite/ld-x86-64/pr23194.d: Likewise. + * testsuite/ld-x86-64/pr23194.map: Likewise. + * testsuite/ld-x86-64/pr23194.s: Likewise. + +2018-05-18 H.J. Lu + * testsuite/ld-i386/i386.exp: Run pr23189 for all targets. * testsuite/ld-x86-64/x86-64.exp: Likewise. diff --git a/ld/testsuite/ld-i386/i386.exp b/ld/testsuite/ld-i386/i386.exp index 5683a75..bb91a76 100644 --- a/ld/testsuite/ld-i386/i386.exp +++ b/ld/testsuite/ld-i386/i386.exp @@ -459,6 +459,7 @@ run_dump_test "pr22135" run_dump_test "pr22782" run_dump_test "pr22929" run_dump_test "pr23189" +run_dump_test "pr23194" if { !([istarget "i?86-*-linux*"] || [istarget "i?86-*-gnu*"] diff --git a/ld/testsuite/ld-i386/pr23194.d b/ld/testsuite/ld-i386/pr23194.d new file mode 100644 index 0000000..b5e8c33 --- /dev/null +++ b/ld/testsuite/ld-i386/pr23194.d @@ -0,0 +1,7 @@ +#as: --32 -mrelax-relocations=yes +#ld: -shared -melf_i386 --version-script pr23194.map +#readelf: -r --wide + +Relocation section '.rel.dyn' at offset 0x[0-9a-f]+ contains 1 entry: + Offset Info Type Sym. Value Symbol's Name +[0-9a-f]+ +[0-9a-f]+ +R_386_RELATIVE + diff --git a/ld/testsuite/ld-i386/pr23194.map b/ld/testsuite/ld-i386/pr23194.map new file mode 100644 index 0000000..353c194 --- /dev/null +++ b/ld/testsuite/ld-i386/pr23194.map @@ -0,0 +1,6 @@ +FOO { + global: + bar; + local: + *; +}; diff --git a/ld/testsuite/ld-i386/pr23194.s b/ld/testsuite/ld-i386/pr23194.s new file mode 100644 index 0000000..32693fe --- /dev/null +++ b/ld/testsuite/ld-i386/pr23194.s @@ -0,0 +1,12 @@ + .text + .symver foo,foo@FOO + .globl foo + .type foo, @function +foo: + ret + .size foo, .-foo + .globl bar + .type bar, @function +bar: + jmp *foo@GOT(%eax) + .size bar, .-bar diff --git a/ld/testsuite/ld-x86-64/pr23194.d b/ld/testsuite/ld-x86-64/pr23194.d new file mode 100644 index 0000000..6b12060 --- /dev/null +++ b/ld/testsuite/ld-x86-64/pr23194.d @@ -0,0 +1,7 @@ +#as: --64 -mrelax-relocations=yes +#ld: -shared -melf_x86_64 --version-script pr23194.map +#readelf: -r --wide + +Relocation section '.rela.dyn' at offset 0x[0-9a-f]+ contains 1 entry: + Offset Info Type Symbol's Value Symbol's Name \+ Addend +[0-9a-f]+ +[0-9a-f]+ +R_X86_64_RELATIVE +[0-9a-f]+ diff --git a/ld/testsuite/ld-x86-64/pr23194.map b/ld/testsuite/ld-x86-64/pr23194.map new file mode 100644 index 0000000..353c194 --- /dev/null +++ b/ld/testsuite/ld-x86-64/pr23194.map @@ -0,0 +1,6 @@ +FOO { + global: + bar; + local: + *; +}; diff --git a/ld/testsuite/ld-x86-64/pr23194.s b/ld/testsuite/ld-x86-64/pr23194.s new file mode 100644 index 0000000..277c19d --- /dev/null +++ b/ld/testsuite/ld-x86-64/pr23194.s @@ -0,0 +1,12 @@ + .text + .symver foo,foo@FOO + .globl foo + .type foo, @function +foo: + ret + .size foo, .-foo + .globl bar + .type bar, @function +bar: + jmp *foo@GOTPCREL(%rip) + .size bar, .-bar diff --git a/ld/testsuite/ld-x86-64/x86-64.exp b/ld/testsuite/ld-x86-64/x86-64.exp index 9654cda..d07054c 100644 --- a/ld/testsuite/ld-x86-64/x86-64.exp +++ b/ld/testsuite/ld-x86-64/x86-64.exp @@ -396,6 +396,7 @@ run_dump_test "pr22135" run_dump_test "pr22782a" run_dump_test "pr22782b" run_dump_test "pr23189" +run_dump_test "pr23194" if { ![istarget "x86_64-*-linux*"] && ![istarget "x86_64-*-nacl*"]} { return