diff options
| author | Owen Anderson <resistor@mac.com> | 2009-05-29 05:37:54 +0000 |
|---|---|---|
| committer | Owen Anderson <resistor@mac.com> | 2009-05-29 05:37:54 +0000 |
| commit | 04cfdd38a26620e509bd852afd993b3dd458dc33 (patch) | |
| tree | ee2acd2d602994065d48299d5ad4afc08b84b3f7 /llvm/lib/Transforms | |
| parent | 87b037168e4cf277eff4c2b1ff67f733986e787f (diff) | |
| download | bcm5719-llvm-04cfdd38a26620e509bd852afd993b3dd458dc33.tar.gz bcm5719-llvm-04cfdd38a26620e509bd852afd993b3dd458dc33.zip | |
Fix an issue where phiMap was not being updated properly when doing load PRE.
Diagnosis and patch thanks to Jakub Staszak.
llvm-svn: 72562
Diffstat (limited to 'llvm/lib/Transforms')
| -rw-r--r-- | llvm/lib/Transforms/Scalar/GVN.cpp | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/llvm/lib/Transforms/Scalar/GVN.cpp b/llvm/lib/Transforms/Scalar/GVN.cpp index 260bbedddc9..222320f5257 100644 --- a/llvm/lib/Transforms/Scalar/GVN.cpp +++ b/llvm/lib/Transforms/Scalar/GVN.cpp @@ -1125,6 +1125,11 @@ bool GVN::processNonLocalLoad(LoadInst *LI, LI->getAlignment(), UnavailablePred->getTerminator()); + SmallPtrSet<Instruction*, 4> &p = phiMap[LI->getPointerOperand()]; + for (SmallPtrSet<Instruction*, 4>::iterator I = p.begin(), E = p.end(); + I != E; ++I) + ValuesPerBlock.push_back(std::make_pair((*I)->getParent(), *I)); + DenseMap<BasicBlock*, Value*> BlockReplValues; BlockReplValues.insert(ValuesPerBlock.begin(), ValuesPerBlock.end()); BlockReplValues[UnavailablePred] = NewLoad; |
