[llvm] r282057 - AArch64: Set shift bit of TLSLE HI12 add instruction
Liu, Lei via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 21 18:14:47 PDT 2016
Hi Diana,
Sorry for the breakage. I made a fix for the problem. I think
it is good now. Could you please take a look?
https://reviews.llvm.org/D24702
Thanks,
Lei
On 2016年09月21日 16:35, Diana Picus wrote:
> Hi Lei,
>
> I'm sorry, but I had to revert this because it broke a lot of
> buildbots. Let me know if you need help investigating.
>
> Regards,
> Diana
>
> http://lab.llvm.org:8011/builders/clang-cmake-aarch64-42vma/builds/12063,
> http://lab.llvm.org:8011/builders/clang-ppc64be-linux-lnt/builds/7898
>
> On 21 September 2016 at 10:41, Lei Liu via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
>> Author: lliu0
>> Date: Wed Sep 21 02:41:41 2016
>> New Revision: 282057
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=282057&view=rev
>> Log:
>> AArch64: Set shift bit of TLSLE HI12 add instruction
>>
>> Summary: AArch64 LLVM assembler emits add instruction without shift bit to calculate the higher 12-bit address of TLS variables in local exec model. This generates wrong code sequence to access TLS variables with thread offset larger than 0x1000.
>>
>> Reviewers: t.p.northover, peter.smith, rovka
>>
>> Subscribers: salim.nasser, aemerson, llvm-commits, rengolin
>>
>> Differential Revision: https://reviews.llvm.org/D24702
>>
>> Added:
>> llvm/trunk/test/MC/AArch64/tls-add-shift.s
>> Modified:
>> llvm/trunk/lib/Target/AArch64/MCTargetDesc/AArch64MCCodeEmitter.cpp
>>
>> Modified: llvm/trunk/lib/Target/AArch64/MCTargetDesc/AArch64MCCodeEmitter.cpp
>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AArch64/MCTargetDesc/AArch64MCCodeEmitter.cpp?rev=282057&r1=282056&r2=282057&view=diff
>> ==============================================================================
>> --- llvm/trunk/lib/Target/AArch64/MCTargetDesc/AArch64MCCodeEmitter.cpp (original)
>> +++ llvm/trunk/lib/Target/AArch64/MCTargetDesc/AArch64MCCodeEmitter.cpp Wed Sep 21 02:41:41 2016
>> @@ -263,6 +263,12 @@ AArch64MCCodeEmitter::getAddSubImmOpValu
>>
>> ++MCNumFixups;
>>
>> + // Set the shift bit of the add instruction for relocation types
>> + // R_AARCH64_TLSLE_ADD_TPREL_HI12 and R_AARCH64_TLSLD_ADD_DTPREL_HI12.
>> + AArch64MCExpr::VariantKind RefKind = cast<AArch64MCExpr>(Expr)->getKind();
>> + if (RefKind == AArch64MCExpr::VK_TPREL_HI12 ||
>> + RefKind == AArch64MCExpr::VK_DTPREL_HI12)
>> + ShiftVal = 12;
>> return ShiftVal == 0 ? 0 : (1 << ShiftVal);
>> }
>>
>>
>> Added: llvm/trunk/test/MC/AArch64/tls-add-shift.s
>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/MC/AArch64/tls-add-shift.s?rev=282057&view=auto
>> ==============================================================================
>> --- llvm/trunk/test/MC/AArch64/tls-add-shift.s (added)
>> +++ llvm/trunk/test/MC/AArch64/tls-add-shift.s Wed Sep 21 02:41:41 2016
>> @@ -0,0 +1,12 @@
>> +// RUN: llvm-mc -triple=aarch64-none-linux-gnu -filetype=obj < %s -o - | \
>> +// RUN: llvm-objdump -r -d - | FileCheck %s
>> +
>> + // TLS add TPREL
>> + add x2, x1, #:tprel_hi12:var
>> +// CHECK: add x2, x1, #0, lsl #12
>> +// CHECK-NEXT: R_AARCH64_TLSLE_ADD_TPREL_HI12 var
>> +
>> + // TLS add DTPREL
>> + add x4, x3, #:dtprel_hi12:var
>> +// CHECK: add x4, x3, #0, lsl #12
>> +// CHECK-NEXT: R_AARCH64_TLSLD_ADD_DTPREL_HI12 var
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
More information about the llvm-commits
mailing list