RPCSX/llvm
1
0
Fork 0
mirror of https://github.com/RPCSX/llvm.git synced 2024-12-12 06:06:19 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix bug: test/Regression/Transforms/SCCP/2002-05-21-InvalidSimplify.ll

Browse Source 
Improperly handling edges... by not marking them alive properly


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@2707 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Chris Lattner 2002-05-22 16:07:20 +00:00
parent bec7dcad14
commit 7f9412b50d
1 changed files with 12 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
lib/Transforms/Scalar/SCCP.cpp
View File

@ -176,16 +176,6 @@ private:
DEBUG_SCCP(cerr << "Marking BB Executable: " << BB); DEBUG_SCCP(cerr << "Marking BB Executable: " << BB);
BBExecutable.insert(BB); // Basic block is executable! BBExecutable.insert(BB); // Basic block is executable!
BBWorkList.push_back(BB); // Add the block to the work list! BBWorkList.push_back(BB); // Add the block to the work list!
// Visit all of the PHI nodes that merge values from this block... Because
// this block is newly executable, PHI nodes that used to be constant now
// may not be. Note that we only mark PHI nodes that live in blocks that
// can execute!
//
for (Value::use_iterator I = BB->use_begin(), E = BB->use_end(); I != E;++I)
if (PHINode *PN = dyn_cast<PHINode>(*I))
if (BBExecutable.count(PN->getParent()))
visitPHINode(PN);
} }
@ -487,8 +477,18 @@ void SCCP::visitTerminatorInst(TerminatorInst *TI) {
// Mark all feasible successors executable... // Mark all feasible successors executable...
for (unsigned i = 0, e = SuccFeasible.size(); i != e; ++i) for (unsigned i = 0, e = SuccFeasible.size(); i != e; ++i)
if (SuccFeasible[i]) if (SuccFeasible[i]) {
markExecutable(TI->getSuccessor(i)); BasicBlock *Succ = TI->getSuccessor(i);
markExecutable(Succ);
// Visit all of the PHI nodes that merge values from this block...
// Because this edge may be new executable, and PHI nodes that used to be
// constant now may not be.
//
for (BasicBlock::iterator I = Succ->begin();
PHINode *PN = dyn_cast<PHINode>(*I); ++I)
visitPHINode(PN);
}
} }
void SCCP::visitUnaryOperator(Instruction *I) { void SCCP::visitUnaryOperator(Instruction *I) {