AArch64: Fix LD crash on weak and undefined TLS symbols. (PR/24602).
authorTamar Christina <tamar.christina@arm.com>
Thu, 22 Aug 2019 10:35:35 +0000 (11:35 +0100)
committerTamar Christina <tamar.christina@arm.com>
Thu, 22 Aug 2019 10:43:15 +0000 (11:43 +0100)
commit652afeef247770b22c44ca292d1f4c65be40a696
tree67dbfe62d2b3c87351b844f745f4363b1b0bb512
parentb20d385926d84f7daa8cd7c54f748f3d6a81bfc6
AArch64: Fix LD crash on weak and undefined TLS symbols. (PR/24602).

This patch fixes a few linker crashes due to TLS code reaching an assert when it
shouldn't.

The first scenario is with weak TLS symbols that remain weak during linking.  In
this case the mid-end would not have seen a TLS symbol and so wouldn't have
allocated the TLS section.  We currently assert here and the linker crashes with
a not very useful message.

This patch changes this to return the value 0 for the TLS symbol in question
emulating what lld and gold and other BFD targets do.  However because weak TLS
is implementation defined and we don't define any behavior for it I also emit a
warning to the user to inform them of such.

Secondly when a strong TLS reference is undefined. The linker crashes even after
it correctly reported that there is an undefined reference.  This changes it so
that it gracefully exits and reports a useful error.

bfd/ChangeLog:

PR ld/24601
* elfnn-aarch64.c (aarch64_relocate): Handle weak TLS and undefined TLS.
Also Pass input_bfd to _bfd_aarch64_elf_resolve_relocation.
* elfxx-aarch64.c (_bfd_aarch64_elf_resolve_relocation): Use it.
* elfxx-aarch64.h (_bfd_aarch64_elf_resolve_relocation): Emit warning
for weak TLS.

ld/ChangeLog:

PR ld/24601
* testsuite/ld-aarch64/aarch64-elf.exp (undef-tls, weak-tls): New.
* testsuite/ld-aarch64/undef-tls.d: New test.
* testsuite/ld-aarch64/undef-tls.s: New test.
* testsuite/ld-aarch64/weak-tls.d: New test.
* testsuite/ld-aarch64/weak-tls.s: New test.
bfd/ChangeLog
bfd/elfnn-aarch64.c
bfd/elfxx-aarch64.c
bfd/elfxx-aarch64.h
ld/ChangeLog
ld/testsuite/ld-aarch64/aarch64-elf.exp
ld/testsuite/ld-aarch64/undef-tls.d [new file with mode: 0644]
ld/testsuite/ld-aarch64/undef-tls.s [new file with mode: 0644]
ld/testsuite/ld-aarch64/weak-tls.d [new file with mode: 0644]
ld/testsuite/ld-aarch64/weak-tls.s [new file with mode: 0644]
This page took 0.026442 seconds and 4 git commands to generate.