diff options
author | Dan Gohman <gohman@apple.com> | 2009-05-04 19:50:33 +0000 |
---|---|---|
committer | Dan Gohman <gohman@apple.com> | 2009-05-04 19:50:33 +0000 |
commit | 7d04e4a7c0a803bc7ab04986ad184291338f42da (patch) | |
tree | f17a94c7b843d5d70a58a372982558c4dc3c776a | |
parent | 28f1412c97fc0740d6743c5226a4ddd65b72c891 (diff) | |
download | llvm-7d04e4a7c0a803bc7ab04986ad184291338f42da.tar.gz llvm-7d04e4a7c0a803bc7ab04986ad184291338f42da.tar.bz2 llvm-7d04e4a7c0a803bc7ab04986ad184291338f42da.tar.xz |
X86FastISel doesn't support the -tailcallopt ABI.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@70902 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/Target/X86/X86FastISel.cpp | 6 | ||||
-rw-r--r-- | test/CodeGen/X86/fast-isel-tailcall.ll | 13 |
2 files changed, 19 insertions, 0 deletions
diff --git a/lib/Target/X86/X86FastISel.cpp b/lib/Target/X86/X86FastISel.cpp index 2a44803f5a..811887bdc6 100644 --- a/lib/Target/X86/X86FastISel.cpp +++ b/lib/Target/X86/X86FastISel.cpp @@ -30,6 +30,7 @@ #include "llvm/CodeGen/MachineRegisterInfo.h" #include "llvm/Support/CallSite.h" #include "llvm/Support/GetElementPtrTypeIterator.h" +#include "llvm/Target/TargetOptions.h" using namespace llvm; namespace { @@ -1115,6 +1116,11 @@ bool X86FastISel::X86SelectCall(Instruction *I) { CC != CallingConv::X86_FastCall) return false; + // On X86, -tailcallopt changes the fastcc ABI. FastISel doesn't + // handle this for now. + if (CC == CallingConv::Fast && PerformTailCallOpt) + return false; + // Let SDISel handle vararg functions. const PointerType *PT = cast<PointerType>(CS.getCalledValue()->getType()); const FunctionType *FTy = cast<FunctionType>(PT->getElementType()); diff --git a/test/CodeGen/X86/fast-isel-tailcall.ll b/test/CodeGen/X86/fast-isel-tailcall.ll new file mode 100644 index 0000000000..6f4d202681 --- /dev/null +++ b/test/CodeGen/X86/fast-isel-tailcall.ll @@ -0,0 +1,13 @@ +; RUN: llvm-as < %s | llc -fast-isel -tailcallopt -march=x86 | not grep add +; PR4154 + +; On x86, -tailcallopt changes the ABI so the caller shouldn't readjust +; the stack pointer after the call in this code. + +define i32 @stub(i8* %t0) nounwind { +entry: + %t1 = load i32* inttoptr (i32 139708680 to i32*) ; <i32> [#uses=1] + %t2 = bitcast i8* %t0 to i32 (i32)* ; <i32 (i32)*> [#uses=1] + %t3 = call fastcc i32 %t2(i32 %t1) ; <i32> [#uses=1] + ret i32 %t3 +} |