diff options
| author | Dan Gohman <gohman@apple.com> | 2009-05-06 17:22:41 +0000 |
|---|---|---|
| committer | Dan Gohman <gohman@apple.com> | 2009-05-06 17:22:41 +0000 |
| commit | e9d87f49063cb1bd213d8e9c339b9b63393cc2d9 (patch) | |
| tree | 80127baa11f20a0ea35766d10184f71f466018a9 /lib/Transforms/Utils/SimplifyCFG.cpp | |
| parent | 0ec4146db70916c31281e091bb3b447aab497246 (diff) | |
Simplify code by using SmallVector's pop_back_val() instead of
separate back() and pop_back() calls.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@71089 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms/Utils/SimplifyCFG.cpp')
| -rw-r--r-- | lib/Transforms/Utils/SimplifyCFG.cpp | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/lib/Transforms/Utils/SimplifyCFG.cpp b/lib/Transforms/Utils/SimplifyCFG.cpp index 1cbf91fd7a..4435b13e00 100644 --- a/lib/Transforms/Utils/SimplifyCFG.cpp +++ b/lib/Transforms/Utils/SimplifyCFG.cpp @@ -742,8 +742,7 @@ static bool FoldValueComparisonIntoPredecessors(TerminatorInst *TI) { SmallVector<BasicBlock*, 16> Preds(pred_begin(BB), pred_end(BB)); while (!Preds.empty()) { - BasicBlock *Pred = Preds.back(); - Preds.pop_back(); + BasicBlock *Pred = Preds.pop_back_val(); // See if the predecessor is a comparison with the same value. TerminatorInst *PTI = Pred->getTerminator(); @@ -1805,10 +1804,9 @@ bool llvm::SimplifyCFG(BasicBlock *BB) { // If we found some, do the transformation! if (!UncondBranchPreds.empty()) { while (!UncondBranchPreds.empty()) { - BasicBlock *Pred = UncondBranchPreds.back(); + BasicBlock *Pred = UncondBranchPreds.pop_back_val(); DOUT << "FOLDING: " << *BB << "INTO UNCOND BRANCH PRED: " << *Pred; - UncondBranchPreds.pop_back(); Instruction *UncondBranch = Pred->getTerminator(); // Clone the return and add it to the end of the predecessor. Instruction *NewRet = RI->clone(); @@ -1847,8 +1845,7 @@ bool llvm::SimplifyCFG(BasicBlock *BB) { // instruction. If any of them just select between returns, change the // branch itself into a select/return pair. while (!CondBranchPreds.empty()) { - BranchInst *BI = CondBranchPreds.back(); - CondBranchPreds.pop_back(); + BranchInst *BI = CondBranchPreds.pop_back_val(); // Check to see if the non-BB successor is also a return block. if (isa<ReturnInst>(BI->getSuccessor(0)->getTerminator()) && |