remove DebugIterations option. Despite the accusations,
jump threading has been shown to only expose problems not
have bugs itself. I'm sure it's completely bug free! ;-)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@60725 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/Transforms/Scalar/JumpThreading.cpp b/lib/Transforms/Scalar/JumpThreading.cpp
index 8dc1ce9..59af14b 100644
--- a/lib/Transforms/Scalar/JumpThreading.cpp
+++ b/lib/Transforms/Scalar/JumpThreading.cpp
@@ -36,11 +36,6 @@
cl::desc("Max block size to duplicate for jump threading"),
cl::init(6), cl::Hidden);
-static cl::opt<int>
-DebugIterations("jump-threading-debug",
- cl::desc("Stop jump threading after N iterations"),
- cl::init(-1), cl::Hidden);
-
namespace {
/// This pass performs 'jump threading', which looks at blocks that have
/// multiple predecessors and multiple successors. If one or more of the
@@ -110,15 +105,11 @@
// If the block is trivially dead, zap it. This eliminates the successor
// edges which simplifies the CFG.
if (pred_begin(BB) == pred_end(BB) &&
- BB != &BB->getParent()->getEntryBlock() &&
- DebugIterations != 0) {
+ BB != &BB->getParent()->getEntryBlock()) {
DOUT << " JT: Deleting dead block '" << BB->getNameStart()
<< "' with terminator: " << *BB->getTerminator();
DeleteDeadBlock(BB);
Changed = true;
-
- if (DebugIterations != -1)
- DebugIterations = DebugIterations-1;
}
}
AnotherIteration = Changed;
@@ -193,10 +184,6 @@
/// ProcessBlock - If there are any predecessors whose control can be threaded
/// through to a successor, transform them now.
bool JumpThreading::ProcessBlock(BasicBlock *BB) {
- if (DebugIterations == 0) return false;
- if (DebugIterations != -1)
- DebugIterations = DebugIterations-1;
-
// If this block has a single predecessor, and if that pred has a single
// successor, merge the blocks. This encourages recursive jump threading
// because now the condition in this block can be threaded through