diff options
| author | Chris Lattner <sabre@nondot.org> | 2008-12-08 22:44:07 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2008-12-08 22:44:07 +0000 |
| commit | e598370ae9638f198d718d9dab724d64a7f37718 (patch) | |
| tree | 5e0cc83ab07f0d8a391dfd00e7d57ebcac880523 /llvm | |
| parent | 223005f35a2408f37d79151bc412167428fdee28 (diff) | |
| download | bcm5719-llvm-e598370ae9638f198d718d9dab724d64a7f37718.tar.gz bcm5719-llvm-e598370ae9638f198d718d9dab724d64a7f37718.zip | |
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! ;-)
llvm-svn: 60725
Diffstat (limited to 'llvm')
| -rw-r--r-- | llvm/lib/Transforms/Scalar/JumpThreading.cpp | 15 |
1 files changed, 1 insertions, 14 deletions
diff --git a/llvm/lib/Transforms/Scalar/JumpThreading.cpp b/llvm/lib/Transforms/Scalar/JumpThreading.cpp index 8dc1ce97f37..59af14b8655 100644 --- a/llvm/lib/Transforms/Scalar/JumpThreading.cpp +++ b/llvm/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 |

