diff options
author | Chad Rosier <mcrosier@apple.com> | 2011-05-20 00:59:28 +0000 |
---|---|---|
committer | Chad Rosier <mcrosier@apple.com> | 2011-05-20 00:59:28 +0000 |
commit | a166089f36cf17cd3733c98000bbe2428011503b (patch) | |
tree | 8fc973cc577f3fcc3523358984c89a7fdcf9e8f1 | |
parent | 2e6496026f41d2c05ff038d14df9972f8a27fb94 (diff) | |
download | llvm-a166089f36cf17cd3733c98000bbe2428011503b.tar.gz llvm-a166089f36cf17cd3733c98000bbe2428011503b.tar.bz2 llvm-a166089f36cf17cd3733c98000bbe2428011503b.tar.xz |
Don't attempt to tail call optimize for Win64.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@131709 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/Target/X86/X86ISelLowering.cpp | 13 | ||||
-rw-r--r-- | test/CodeGen/X86/vararg_tailcall.ll | 4 |
2 files changed, 9 insertions, 8 deletions
diff --git a/lib/Target/X86/X86ISelLowering.cpp b/lib/Target/X86/X86ISelLowering.cpp index e5156f8d41..ce1dc09760 100644 --- a/lib/Target/X86/X86ISelLowering.cpp +++ b/lib/Target/X86/X86ISelLowering.cpp @@ -2531,17 +2531,18 @@ X86TargetLowering::IsEligibleForTailCallOptimization(SDValue Callee, return false; // Do not sibcall optimize vararg calls unless all arguments are passed via - // registers + // registers. if (isVarArg && !Outs.empty()) { + + // Optimizing for varargs on Win64 is unlikely to be safe without + // additional testing. + if (Subtarget->isTargetWin64()) + return false; + SmallVector<CCValAssign, 16> ArgLocs; CCState CCInfo(CalleeCC, isVarArg, getTargetMachine(), ArgLocs, *DAG.getContext()); - // Allocate shadow area for Win64 - if (Subtarget->isTargetWin64()) { - CCInfo.AllocateStack(32, 8); - } - CCInfo.AnalyzeCallOperands(Outs, CC_X86); for (unsigned i = 0, e = ArgLocs.size(); i != e; ++i) if (!ArgLocs[i].isRegLoc()) diff --git a/test/CodeGen/X86/vararg_tailcall.ll b/test/CodeGen/X86/vararg_tailcall.ll index d50b83acc8..73d80ebc1d 100644 --- a/test/CodeGen/X86/vararg_tailcall.ll +++ b/test/CodeGen/X86/vararg_tailcall.ll @@ -12,7 +12,7 @@ ; X64: @foo ; X64: jmp ; WIN64: @foo -; WIN64: jmp +; WIN64: callq define void @foo(i64 %arg) nounwind optsize ssp noredzone { entry: %call = tail call i32 (i8*, ...)* @printf(i8* getelementptr inbounds ([5 x i8]* @.str, i64 0, i64 0), i64 %arg) nounwind optsize noredzone @@ -36,7 +36,7 @@ declare void @bar2(i8*, i64) optsize noredzone ; X64: @foo2 ; X64: jmp ; WIN64: @foo2 -; WIN64: jmp +; WIN64: callq define i8* @foo2(i8* %arg) nounwind optsize ssp noredzone { entry: %tmp1 = load i8** @sel, align 8, !tbaa !0 |