[PATCH] D52525: [AArch64] Fix range check of R_AARCH64_TLSLE_ADD_TPREL_HI12
Ryan Prichard via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 25 17:31:56 PDT 2018
rprichard created this revision.
Herald added subscribers: llvm-commits, kristof.beyls, arichardson, emaste.
Herald added a reviewer: javed.absar.
Herald added a reviewer: espindola.
An AArch64 LE relocation is a positive ("variant 1") offset. This
relocation is used to write the upper 12 bits of a 24-bit offset into an
add instruction:
add x0, x0, :tprel_hi12:v1
The comment in the ARM docs for R_AARCH64_TLSLE_ADD_TPREL_HI12 is:
"Set an ADD immediate field to bits [23:12] of X; check 0 <= X < 224."
Repository:
rLLD LLVM Linker
https://reviews.llvm.org/D52525
Files:
ELF/Arch/AArch64.cpp
test/ELF/aarch64-tls-le.s
Index: test/ELF/aarch64-tls-le.s
===================================================================
--- test/ELF/aarch64-tls-le.s
+++ test/ELF/aarch64-tls-le.s
@@ -13,19 +13,36 @@
mrs x0, TPIDR_EL0
add x0, x0, :tprel_hi12:v1
add x0, x0, :tprel_lo12_nc:v1
+ mrs x0, TPIDR_EL0
+ add x0, x0, :tprel_hi12:v2
+ add x0, x0, :tprel_lo12_nc:v2
# TCB size = 0x16 and foo is first element from TLS register.
#CHECK: Disassembly of section .text:
#CHECK: _start:
#CHECK: 210000: 40 d0 3b d5 mrs x0, TPIDR_EL0
#CHECK: 210004: 00 00 40 91 add x0, x0, #0, lsl #12
#CHECK: 210008: 00 40 00 91 add x0, x0, #16
+#CHECK: 21000c: 40 d0 3b d5 mrs x0, TPIDR_EL0
+#CHECK: 210010: 00 fc 7f 91 add x0, x0, #4095, lsl #12
+#CHECK: 210014: 00 e0 3f 91 add x0, x0, #4088
-.type v1, at object
.section .tbss,"awT", at nobits
+
+.type v1, at object
.globl v1
.p2align 2
v1:
.word 0
.size v1, 4
+# The current offset from the thread pointer is 20. Raise it to just below the
+# 24-bit limit.
+.space (0xfffff8 - 20)
+
+.type v2, at object
+.globl v2
+.p2align 2
+v2:
+.word 0
+.size v2, 4
Index: ELF/Arch/AArch64.cpp
===================================================================
--- ELF/Arch/AArch64.cpp
+++ ELF/Arch/AArch64.cpp
@@ -345,7 +345,7 @@
or32le(Loc, (Val & 0xFFFC) << 3);
break;
case R_AARCH64_TLSLE_ADD_TPREL_HI12:
- checkInt(Loc, Val, 24, Type);
+ checkUInt(Loc, Val, 24, Type);
or32AArch64Imm(Loc, Val >> 12);
break;
case R_AARCH64_TLSLE_ADD_TPREL_LO12_NC:
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52525.167033.patch
Type: text/x-patch
Size: 1565 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180926/5a6186b5/attachment.bin>
More information about the llvm-commits
mailing list