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 | 1c057ee3b353a8c80d7bd9e2d4ea6526d88c7b05 (patch) | |
| tree | 59e02f796946ed8fd3f45005f5752f8a59b9e465 | |
| parent | 28a0fc9876b0e562bcadde90fb35660def5e1ab5 (diff) | |
| download | external_llvm-1c057ee3b353a8c80d7bd9e2d4ea6526d88c7b05.zip external_llvm-1c057ee3b353a8c80d7bd9e2d4ea6526d88c7b05.tar.gz external_llvm-1c057ee3b353a8c80d7bd9e2d4ea6526d88c7b05.tar.bz2 | |
Fix an issue where phiMap was not being updated properly when doing load PRE.
Diagnosis and patch thanks to Jakub Staszak.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@72562 91177308-0d34-0410-b5e6-96231b3b80d8
| -rw-r--r-- | lib/Transforms/Scalar/GVN.cpp | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/lib/Transforms/Scalar/GVN.cpp b/lib/Transforms/Scalar/GVN.cpp index 260bbed..222320f 100644 --- a/lib/Transforms/Scalar/GVN.cpp +++ b/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; |