diff options
author | Vlad Tsyrklevich <vlad@tsyrklevich.net> | 2018-08-01 17:44:37 +0000 |
---|---|---|
committer | Vlad Tsyrklevich <vlad@tsyrklevich.net> | 2018-08-01 17:44:37 +0000 |
commit | ab016e00ecd33b2d873bac4b77c59be0c3e12db3 (patch) | |
tree | d36aea233f1113b671b17f840f2d5d777c0150a4 | |
parent | 931ebe3be185b00a10588e8f36787df7378a0eaf (diff) | |
download | bcm5719-llvm-ab016e00ecd33b2d873bac4b77c59be0c3e12db3.tar.gz bcm5719-llvm-ab016e00ecd33b2d873bac4b77c59be0c3e12db3.zip |
[X86] FastISel fall back on !absolute_symbol GVs
Summary:
D25878, which added support for !absolute_symbol for normal X86 ISel,
did not add support for materializing references to absolute symbols for
X86 FastISel. This causes build failures because FastISel generates
PC-relative relocations for absolute symbols. Fall back to normal ISel
for references to !absolute_symbol GVs. Fix for PR38200.
Reviewers: pcc, craig.topper
Reviewed By: pcc
Subscribers: hiraditya, llvm-commits, kcc
Differential Revision: https://reviews.llvm.org/D50116
llvm-svn: 338599
-rw-r--r-- | llvm/lib/Target/X86/X86FastISel.cpp | 4 | ||||
-rw-r--r-- | llvm/test/CodeGen/X86/absolute-bit-mask-fastisel.ll | 28 |
2 files changed, 32 insertions, 0 deletions
diff --git a/llvm/lib/Target/X86/X86FastISel.cpp b/llvm/lib/Target/X86/X86FastISel.cpp index 35a15577fe0..d082b42eefa 100644 --- a/llvm/lib/Target/X86/X86FastISel.cpp +++ b/llvm/lib/Target/X86/X86FastISel.cpp @@ -738,6 +738,10 @@ bool X86FastISel::handleConstantAddresses(const Value *V, X86AddressMode &AM) { if (GV->isThreadLocal()) return false; + // Can't handle !absolute_symbol references yet. + if (GV->isAbsoluteSymbolRef()) + return false; + // RIP-relative addresses can't have additional register operands, so if // we've already folded stuff into the addressing mode, just force the // global value into its own register, which we can use as the basereg. diff --git a/llvm/test/CodeGen/X86/absolute-bit-mask-fastisel.ll b/llvm/test/CodeGen/X86/absolute-bit-mask-fastisel.ll new file mode 100644 index 00000000000..34e92e80457 --- /dev/null +++ b/llvm/test/CodeGen/X86/absolute-bit-mask-fastisel.ll @@ -0,0 +1,28 @@ +; RUN: llc < %s | FileCheck %s +; RUN: llc -relocation-model=pic < %s | FileCheck %s + +; Regression test for PR38200 + +target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128" +target triple = "x86_64-unknown-linux-gnu" + +@bit_mask8 = external hidden global i8, !absolute_symbol !0 + +declare void @f() + +define void @foo8(i8* %ptr) noinline optnone { + %load = load i8, i8* %ptr + ; CHECK: movl $bit_mask8, %ecx + %and = and i8 %load, ptrtoint (i8* @bit_mask8 to i8) + %icmp = icmp eq i8 %and, 0 + br i1 %icmp, label %t, label %f + +t: + call void @f() + ret void + +f: + ret void +} + +!0 = !{i64 0, i64 256} |