make sure to set Changed=true when instcombine hacks on the code,

not doing so prevents it from properly iterating and prevents it
from deleting the entire body of dce-iterate.ll


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@63476 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Chris Lattner
2009-01-31 07:04:22 +00:00
parent 95be699a9a
commit 1e19d603e0
2 changed files with 33 additions and 5 deletions

View File

@@ -12356,6 +12356,7 @@ bool InstCombiner::DoOneIteration(Function &F, unsigned Iteration) {
if (!I->use_empty())
I->replaceAllUsesWith(UndefValue::get(I->getType()));
I->eraseFromParent();
Changed = true;
}
}
}
@@ -12375,6 +12376,7 @@ bool InstCombiner::DoOneIteration(Function &F, unsigned Iteration) {
I->eraseFromParent();
RemoveFromWorkList(I);
Changed = true;
continue;
}
@@ -12389,17 +12391,19 @@ bool InstCombiner::DoOneIteration(Function &F, unsigned Iteration) {
++NumConstProp;
I->eraseFromParent();
RemoveFromWorkList(I);
Changed = true;
continue;
}
if (TD && I->getType()->getTypeID() == Type::VoidTyID) {
// See if we can constant fold its operands.
for (User::op_iterator i = I->op_begin(), e = I->op_end(); i != e; ++i) {
if (ConstantExpr *CE = dyn_cast<ConstantExpr>(i)) {
for (User::op_iterator i = I->op_begin(), e = I->op_end(); i != e; ++i)
if (ConstantExpr *CE = dyn_cast<ConstantExpr>(i))
if (Constant *NewC = ConstantFoldConstantExpression(CE, TD))
i->set(NewC);
}
}
if (NewC != CE) {
i->set(NewC);
Changed = true;
}
}
// See if we can trivially sink this instruction to a successor basic block.