diff options
author | Nate Begeman <natebegeman@mac.com> | 2008-04-25 06:37:06 +0000 |
---|---|---|
committer | Nate Begeman <natebegeman@mac.com> | 2008-04-25 06:37:06 +0000 |
commit | 6fed3b20389ef0a9dbdd8bf7d9fd686fdd8b90f6 (patch) | |
tree | 87256584cc00b835301055b64b1d99a34cb3fc91 | |
parent | 741c7a3b493fe9508c675baca2529302d80df672 (diff) | |
download | bcm5719-llvm-6fed3b20389ef0a9dbdd8bf7d9fd686fdd8b90f6.tar.gz bcm5719-llvm-6fed3b20389ef0a9dbdd8bf7d9fd686fdd8b90f6.zip |
Teach the PruningFunctionCloner how to look through loads with
ConstantExpression GEPs pointing into constant globals.
llvm-svn: 50256
-rw-r--r-- | llvm/lib/Transforms/Utils/CloneFunction.cpp | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/llvm/lib/Transforms/Utils/CloneFunction.cpp b/llvm/lib/Transforms/Utils/CloneFunction.cpp index ba9b57d4f56..ca63399c811 100644 --- a/llvm/lib/Transforms/Utils/CloneFunction.cpp +++ b/llvm/lib/Transforms/Utils/CloneFunction.cpp @@ -17,6 +17,7 @@ #include "llvm/Constants.h" #include "llvm/DerivedTypes.h" #include "llvm/Instructions.h" +#include "llvm/GlobalVariable.h" #include "llvm/Function.h" #include "llvm/Support/CFG.h" #include "llvm/Support/Compiler.h" @@ -308,13 +309,20 @@ ConstantFoldMappedInstruction(const Instruction *I) { else return 0; // All operands not constant! - if (const CmpInst *CI = dyn_cast<CmpInst>(I)) return ConstantFoldCompareInstOperands(CI->getPredicate(), &Ops[0], Ops.size(), TD); - else - return ConstantFoldInstOperands(I->getOpcode(), I->getType(), - &Ops[0], Ops.size(), TD); + + if (ConstantExpr *CE = dyn_cast<ConstantExpr>(Ops[0])) + if (const LoadInst *LI = dyn_cast<LoadInst>(I)) + if (!LI->isVolatile() && CE->getOpcode() == Instruction::GetElementPtr) + if (GlobalVariable *GV = dyn_cast<GlobalVariable>(CE->getOperand(0))) + if (GV->isConstant() && !GV->isDeclaration()) + return ConstantFoldLoadThroughGEPConstantExpr(GV->getInitializer(), + CE); + + return ConstantFoldInstOperands(I->getOpcode(), I->getType(), &Ops[0], + Ops.size(), TD); } /// CloneAndPruneFunctionInto - This works exactly like CloneFunctionInto, |