aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChris Lattner <sabre@nondot.org>2008-12-08 22:44:07 +0000
committerChris Lattner <sabre@nondot.org>2008-12-08 22:44:07 +0000
commit20fa76ef6f7c2d3073e0960cf347af8db64708fc (patch)
treefc34e5308e9b0d51cc72080b8b08d16974c9645c
parente4d1082b2c63919f3abb6de18ce47ce9e05cfdea (diff)
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
-rw-r--r--lib/Transforms/Scalar/JumpThreading.cpp15
1 files changed, 1 insertions, 14 deletions
diff --git a/lib/Transforms/Scalar/JumpThreading.cpp b/lib/Transforms/Scalar/JumpThreading.cpp
index 8dc1ce97f3..59af14b865 100644
--- a/lib/Transforms/Scalar/JumpThreading.cpp
+++ b/lib/Transforms/Scalar/JumpThreading.cpp
@@ -36,11 +36,6 @@ Threshold("jump-threading-threshold",
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 @@ bool JumpThreading::runOnFunction(Function &F) {
// 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 @@ static unsigned getJumpThreadDuplicationCost(const BasicBlock *BB) {
/// 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