From 20fa76ef6f7c2d3073e0960cf347af8db64708fc Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Mon, 8 Dec 2008 22:44:07 +0000 Subject: [PATCH] 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 --- lib/Transforms/Scalar/JumpThreading.cpp | 15 +-------------- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/lib/Transforms/Scalar/JumpThreading.cpp b/lib/Transforms/Scalar/JumpThreading.cpp index 8dc1ce97f37..59af14b8655 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 -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