diff options
| author | Chris Lattner <sabre@nondot.org> | 2007-05-19 06:51:32 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2007-05-19 06:51:32 +0000 |
| commit | a2b18de4ba9ac3f5ac14ba9984d2250b0e512ea1 (patch) | |
| tree | b1fe2cb7ee33dc0b8e893f78bc102c692ad4803f | |
| parent | 74b9a814f6b87790f8393413a7f4d1d7debcd088 (diff) | |
Fix Transforms/InstCombine/2007-05-18-CastFoldBug.ll, a bug that devastates
objc code due to the way the FE lowers objc message sends.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@37256 91177308-0d34-0410-b5e6-96231b3b80d8
| -rw-r--r-- | lib/Transforms/Scalar/InstructionCombining.cpp | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/lib/Transforms/Scalar/InstructionCombining.cpp b/lib/Transforms/Scalar/InstructionCombining.cpp index 08905576d2..db105adf7f 100644 --- a/lib/Transforms/Scalar/InstructionCombining.cpp +++ b/lib/Transforms/Scalar/InstructionCombining.cpp @@ -7773,6 +7773,14 @@ bool InstCombiner::transformConstExprCastCall(CallSite CS) { const FunctionType *FT = Callee->getFunctionType(); const Type *OldRetTy = Caller->getType(); + const FunctionType *ActualFT = + cast<FunctionType>(cast<PointerType>(CE->getType())->getElementType()); + + // If the parameter attributes don't match up, don't do the xform. We don't + // want to lose an sret attribute or something. + if (FT->getParamAttrs() != ActualFT->getParamAttrs()) + return false; + // Check to see if we are changing the return type... if (OldRetTy != FT->getReturnType()) { if (Callee->isDeclaration() && !Caller->use_empty() && |