[Committed][AArch64] Add support for :tlsdesc: and TLSDESC_ADR_PREL21
Marcus Shawcroft
marcus.shawcroft@gmail.com
Fri Feb 27 14:31:00 GMT 2015
On 27 February 2015 at 01:37, Alan Modra <amodra@gmail.com> wrote:
> On Thu, Feb 26, 2015 at 10:57:39PM +0000, Marcus Shawcroft wrote:
>> 2015-02-26 Marcus Shawcroft <marcus.shawcroft@arm.com>
>>
>> + * elfnn-aarch64.c (IS_AARCH64_TLSDESC_RELOC) BFD_RELOC_AARCH64_TLSDESC_ADR_PREL21 move.
>
> Please fix this overly long entry, which is missing a colon and is
> somewhat obscure. The long line attracted my attention, then I
> wondered what a "BFD_RELOC... move" was! "Move BFD_RELOC... test"
> would have been clearer. Or just
> * elfnn-aarch64.c (IS_AARCH64_TLSDESC_RELOC): Reorder R_TYPE tests.
Hi, Fixed as suggested, the entry now reads:
2015-02-26 Marcus Shawcroft <marcus.shawcroft@arm.com>
* elfnn-aarch64.c (IS_AARCH64_TLSDESC_RELOC): Reorder R_TYPE
tests.
(aarch64_tls_transition_without_check)
(aarch64_reloc_got_type, elfNN_aarch64_final_link_relocate)
(elfNN_aarch64_tls_relax, elfNN_aarch64_relocate_section)
(elfNN_aarch64_gc_sweep_hook, elfNN_aarch64_check_relocs): Handle
BFD_RELOC_AARCH64_TLSDESC_ADR_PREL21.
* elfxx-aarch64.c (_bfd_aarch64_elf_put_addend)
(_bfd_aarch64_elf_resolve_relocation): Likewise.
* reloc.c (BFD_RELOC_AARCH64_TLSGD_ADR_PREL21): Define.
Cheers
/Marcus
More information about the Binutils
mailing list