diff options
author | Evan Cheng <evan.cheng@apple.com> | 2010-12-01 22:59:46 +0000 |
---|---|---|
committer | Evan Cheng <evan.cheng@apple.com> | 2010-12-01 22:59:46 +0000 |
commit | 1bf891ae6e174ac92c1091be0c62891f4676beb8 (patch) | |
tree | 68f0511c8dbeac9965149c03e126ac828aed90a3 /lib/Target/X86/X86ISelLowering.cpp | |
parent | 6566e0a55e3d9403f77db81a79a5dadedb4e666e (diff) | |
download | external_llvm-1bf891ae6e174ac92c1091be0c62891f4676beb8.zip external_llvm-1bf891ae6e174ac92c1091be0c62891f4676beb8.tar.gz external_llvm-1bf891ae6e174ac92c1091be0c62891f4676beb8.tar.bz2 |
Fix and re-enable tail call optimization of expanded libcalls.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@120622 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target/X86/X86ISelLowering.cpp')
-rw-r--r-- | lib/Target/X86/X86ISelLowering.cpp | 37 |
1 files changed, 19 insertions, 18 deletions
diff --git a/lib/Target/X86/X86ISelLowering.cpp b/lib/Target/X86/X86ISelLowering.cpp index 07bd5cf..f3c1b85 100644 --- a/lib/Target/X86/X86ISelLowering.cpp +++ b/lib/Target/X86/X86ISelLowering.cpp @@ -1339,22 +1339,25 @@ X86TargetLowering::LowerReturn(SDValue Chain, } bool X86TargetLowering::isUsedByReturnOnly(SDNode *N) const { - // Temporarily disabled. - return false; if (N->getNumValues() != 1) return false; if (!N->hasNUsesOfValue(1, 0)) return false; SDNode *Copy = *N->use_begin(); - if (Copy->getOpcode() != ISD::CopyToReg) + if (Copy->getOpcode() != ISD::CopyToReg && + Copy->getOpcode() != ISD::FP_EXTEND) return false; + + bool HasRet = false; for (SDNode::use_iterator UI = Copy->use_begin(), UE = Copy->use_end(); - UI != UE; ++UI) + UI != UE; ++UI) { if (UI->getOpcode() != X86ISD::RET_FLAG) return false; + HasRet = true; + } - return true; + return HasRet; } /// LowerCallResult - Lower the result values of a call into the @@ -2161,19 +2164,17 @@ X86TargetLowering::LowerCall(SDValue Chain, SDValue Callee, } else if (ExternalSymbolSDNode *S = dyn_cast<ExternalSymbolSDNode>(Callee)) { unsigned char OpFlags = 0; - if (!isTailCall) { - // On ELF targets, in either X86-64 or X86-32 mode, direct calls to - // external symbols should go through the PLT. - if (Subtarget->isTargetELF() && - getTargetMachine().getRelocationModel() == Reloc::PIC_) { - OpFlags = X86II::MO_PLT; - } else if (Subtarget->isPICStyleStubAny() && - Subtarget->getDarwinVers() < 9) { - // PC-relative references to external symbols should go through $stub, - // unless we're building with the leopard linker or later, which - // automatically synthesizes these stubs. - OpFlags = X86II::MO_DARWIN_STUB; - } + // On ELF targets, in either X86-64 or X86-32 mode, direct calls to + // external symbols should go through the PLT. + if (Subtarget->isTargetELF() && + getTargetMachine().getRelocationModel() == Reloc::PIC_) { + OpFlags = X86II::MO_PLT; + } else if (Subtarget->isPICStyleStubAny() && + Subtarget->getDarwinVers() < 9) { + // PC-relative references to external symbols should go through $stub, + // unless we're building with the leopard linker or later, which + // automatically synthesizes these stubs. + OpFlags = X86II::MO_DARWIN_STUB; } Callee = DAG.getTargetExternalSymbol(S->getSymbol(), getPointerTy(), |