Remove unneeded iteration. Thanks to Dan for the feedback.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@54337 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/CodeGen/UnreachableBlockElim.cpp b/lib/CodeGen/UnreachableBlockElim.cpp
index 8286239..19341ee 100644
--- a/lib/CodeGen/UnreachableBlockElim.cpp
+++ b/lib/CodeGen/UnreachableBlockElim.cpp
@@ -85,7 +85,6 @@
class VISIBILITY_HIDDEN UnreachableMachineBlockElim :
public MachineFunctionPass {
virtual bool runOnMachineFunction(MachineFunction &F);
- bool iterateOnFunction(MachineFunction& F);
public:
static char ID; // Pass identification, replacement for typeid
@@ -101,21 +100,6 @@
const PassInfo *const llvm::UnreachableMachineBlockElimID = &Y;
bool UnreachableMachineBlockElim::runOnMachineFunction(MachineFunction &F) {
- bool changed = true;
- bool result = false;
-
- while (changed) {
- changed = iterateOnFunction(F);
- result |= changed;
- }
-
- if (result)
- F.RenumberBlocks();
-
- return result;
-}
-
-bool UnreachableMachineBlockElim::iterateOnFunction(MachineFunction &F) {
std::set<MachineBasicBlock*> Reachable;
// Mark all reachable blocks.
@@ -160,6 +144,8 @@
for (unsigned i = 0, e = DeadBlocks.size(); i != e; ++i)
DeadBlocks[i]->eraseFromParent();
+ F.RenumberBlocks();
+
return DeadBlocks.size();
}