diff options
author | Chris Lattner <sabre@nondot.org> | 2008-11-27 23:00:20 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2008-11-27 23:00:20 +0000 |
commit | a0d4486073f6303fbf33b50ac3fbae4601ea0229 (patch) | |
tree | 28506e171c11a2f4caf16b52869eae33fa93d127 /lib/Transforms/Scalar/LoopStrengthReduce.cpp | |
parent | 3481f24c06b3c9de48bdd99c37547471ca8e761e (diff) |
use continue to reduce indentation
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@60192 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms/Scalar/LoopStrengthReduce.cpp')
-rw-r--r-- | lib/Transforms/Scalar/LoopStrengthReduce.cpp | 37 |
1 files changed, 19 insertions, 18 deletions
diff --git a/lib/Transforms/Scalar/LoopStrengthReduce.cpp b/lib/Transforms/Scalar/LoopStrengthReduce.cpp index 9052d6088e..c2f3209f05 100644 --- a/lib/Transforms/Scalar/LoopStrengthReduce.cpp +++ b/lib/Transforms/Scalar/LoopStrengthReduce.cpp @@ -2076,25 +2076,26 @@ bool LoopStrengthReduce::runOnLoop(Loop *L, LPPassManager &LPM) { // // FIXME: this needs to eliminate an induction variable even if it's being // compared against some value to decide loop termination. - if (PN->hasOneUse()) { - SmallPtrSet<PHINode *, 2> PHIs; - for (Instruction *J = dyn_cast<Instruction>(*PN->use_begin()); - J && J->hasOneUse() && !J->mayWriteToMemory(); - J = dyn_cast<Instruction>(*J->use_begin())) { - // If we find the original PHI, we've discovered a cycle. - if (J == PN) { - // Break the cycle and mark the PHI for deletion. - SE->deleteValueFromRecords(PN); - PN->replaceAllUsesWith(UndefValue::get(PN->getType())); - DeadInsts.insert(PN); - Changed = true; - break; - } - // If we find a PHI more than once, we're on a cycle that - // won't prove fruitful. - if (isa<PHINode>(J) && !PHIs.insert(cast<PHINode>(J))) - break; + if (!PN->hasOneUse()) + continue; + + SmallPtrSet<PHINode *, 4> PHIs; + for (Instruction *J = dyn_cast<Instruction>(*PN->use_begin()); + J && J->hasOneUse() && !J->mayWriteToMemory(); + J = dyn_cast<Instruction>(*J->use_begin())) { + // If we find the original PHI, we've discovered a cycle. + if (J == PN) { + // Break the cycle and mark the PHI for deletion. + SE->deleteValueFromRecords(PN); + PN->replaceAllUsesWith(UndefValue::get(PN->getType())); + DeadInsts.insert(PN); + Changed = true; + break; } + // If we find a PHI more than once, we're on a cycle that + // won't prove fruitful. + if (isa<PHINode>(J) && !PHIs.insert(cast<PHINode>(J))) + break; } } DeleteTriviallyDeadInstructions(DeadInsts); |