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 | ab6b226978644c438a3a7768a06dcd48509d000c (patch) | |
tree | 8d574fefe86e8fa7cf23552dc4353c69bc8dec20 /lib/Transforms/Scalar/SCCP.cpp | |
parent | db95fa131a229652f925794ca7a5b84e9490050b (diff) | |
download | external_llvm-ab6b226978644c438a3a7768a06dcd48509d000c.zip external_llvm-ab6b226978644c438a3a7768a06dcd48509d000c.tar.gz external_llvm-ab6b226978644c438a3a7768a06dcd48509d000c.tar.bz2 |
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).
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@67407 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms/Scalar/SCCP.cpp')
-rw-r--r-- | lib/Transforms/Scalar/SCCP.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/Transforms/Scalar/SCCP.cpp b/lib/Transforms/Scalar/SCCP.cpp index 7adc80f..a49bcc8 100644 --- a/lib/Transforms/Scalar/SCCP.cpp +++ b/lib/Transforms/Scalar/SCCP.cpp @@ -1131,7 +1131,7 @@ void SCCPSolver::visitLoadInst(LoadInst &I) { // Transform load (constant global) into the value loaded. if (GlobalVariable *GV = dyn_cast<GlobalVariable>(Ptr)) { if (GV->isConstant()) { - if (!GV->isDeclaration()) { + if (!GV->isDeclaration() && !GV->mayBeOverridden()) { markConstant(IV, &I, GV->getInitializer()); return; } @@ -1150,7 +1150,7 @@ void SCCPSolver::visitLoadInst(LoadInst &I) { if (ConstantExpr *CE = dyn_cast<ConstantExpr>(Ptr)) if (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()) if (Constant *V = ConstantFoldLoadThroughGEPConstantExpr(GV->getInitializer(), CE)) { markConstant(IV, &I, V); |