diff options
author | Han Shen <shenhan@google.com> | 2014-11-26 10:34:46 -0800 |
---|---|---|
committer | Han Shen <shenhan@google.com> | 2014-11-26 10:34:46 -0800 |
commit | bb779192aecf89975aef7a16e1ea67789b5f38dd (patch) | |
tree | c3c230aa3d4caa79a8a62eaa4b9de8ec6fa41f8a /gold/aarch64.cc | |
parent | a11652892c18324bf3abb8b25c01475e5a18632a (diff) | |
download | fsf-binutils-gdb-bb779192aecf89975aef7a16e1ea67789b5f38dd.zip fsf-binutils-gdb-bb779192aecf89975aef7a16e1ea67789b5f38dd.tar.gz fsf-binutils-gdb-bb779192aecf89975aef7a16e1ea67789b5f38dd.tar.bz2 |
Fix for gold linking tlsdesc into an executable with -pie.
(Also included in this patch is a minor typo fix in gold/ChangeLog.)
When linking the following tlsdesc access sequence into an executable with -pie,
adrp x0, :tlsdesc:tls_gd
ldr x1, [x0, #:tlsdesc_lo12:tls_gd]
add x0, x0, :tlsdesc_lo12:tls_gd
.tlsdesccall tls_gd
blr x1
mrs x1, tpidr_el0
add x0, x1, x0
ldr w0, [x0]
current gold-aarch64 backend does tls-desc-gd-to-ie relaxation, into
adrp x0, 1000 <__FRAME_END__+0x720>
ldr x1, [x0,#4064] ;; <=== the target register should be x0
nop
nop
mrs x1, tpidr_el0
add x0, x1, x0
ldr w0, [x0]
This code is wrong. The fix changes ldr target register into x0.
Diffstat (limited to 'gold/aarch64.cc')
-rw-r--r-- | gold/aarch64.cc | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/gold/aarch64.cc b/gold/aarch64.cc index 125953a..d5a0a85 100644 --- a/gold/aarch64.cc +++ b/gold/aarch64.cc @@ -6536,6 +6536,11 @@ Target_aarch64<size, big_endian>::Relocate::tls_desc_gd_to_ie( case elfcpp::R_AARCH64_TLSDESC_LD64_LO12: { + // Set ldr target register to be x0. + Insntype insn = elfcpp::Swap<32, big_endian>::readval(ip); + insn &= 0xffffffe0; + elfcpp::Swap<32, big_endian>::writeval(ip, insn); + // Do relocation. const AArch64_reloc_property* reloc_property = aarch64_reloc_property_table->get_reloc_property( elfcpp::R_AARCH64_TLSIE_LD64_GOTTPREL_LO12_NC); |