diff options
author | Duncan Sands <baldrick@free.fr> | 2009-03-20 21:53:29 +0000 |
---|---|---|
committer | Duncan Sands <baldrick@free.fr> | 2009-03-20 21:53:29 +0000 |
commit | a09e0afe748ce5d14ceb7d948a2660f6360c845e (patch) | |
tree | b99067907ba0148462ee35f858a0d5a65b175131 /llvm/lib/Transforms/Utils/CloneFunction.cpp | |
parent | 692c6e3729149e44ba688a173c42b99c66821aa1 (diff) | |
download | bcm5719-llvm-a09e0afe748ce5d14ceb7d948a2660f6360c845e.tar.gz bcm5719-llvm-a09e0afe748ce5d14ceb7d948a2660f6360c845e.zip |
Don't load values out of global constants with weak
linkage: the value may be replaced with something
different at link time. (Frontends that want to
allow values to be loaded out of weak constants can
give their constants weak_odr linkage).
llvm-svn: 67407
Diffstat (limited to 'llvm/lib/Transforms/Utils/CloneFunction.cpp')
-rw-r--r-- | llvm/lib/Transforms/Utils/CloneFunction.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/Utils/CloneFunction.cpp b/llvm/lib/Transforms/Utils/CloneFunction.cpp index 59211028f31..5dc2d1cdc3a 100644 --- a/llvm/lib/Transforms/Utils/CloneFunction.cpp +++ b/llvm/lib/Transforms/Utils/CloneFunction.cpp @@ -335,7 +335,8 @@ ConstantFoldMappedInstruction(const Instruction *I) { if (ConstantExpr *CE = dyn_cast<ConstantExpr>(Ops[0])) if (!LI->isVolatile() && CE->getOpcode() == Instruction::GetElementPtr) if (GlobalVariable *GV = dyn_cast<GlobalVariable>(CE->getOperand(0))) - if (GV->isConstant() && !GV->isDeclaration()) + if (GV->isConstant() && !GV->isDeclaration() && + !GV->mayBeOverridden()) return ConstantFoldLoadThroughGEPConstantExpr(GV->getInitializer(), CE); |