commit d7708773288c4f33bf98c724f7a486b5973b8cc6
author David Majnemer <david.majnemer@gmail.com> Fri Jun 24 04:05:21 2016 +0000
committer David Majnemer <david.majnemer@gmail.com> Fri Jun 24 04:05:21 2016 +0000
tree da2706e419335edde86f19e16552adcc1da3cd21
parent 024402dcdf0c45b48b15d9a5de7f841e61e9bedc

Switch more loops to be range-based

This makes the code a little more concise, no functional change is
intended.

llvm-svn: 273644

llvm/lib/Transforms/Scalar/MergedLoadStoreMotion.cpp

diff --git a/llvm/lib/Transforms/Scalar/MergedLoadStoreMotion.cpp b/llvm/lib/Transforms/Scalar/MergedLoadStoreMotion.cpp
index e4af991..eb67aea 100644
--- a/llvm/lib/Transforms/Scalar/MergedLoadStoreMotion.cpp
+++ b/llvm/lib/Transforms/Scalar/MergedLoadStoreMotion.cpp
@@ -377,11 +377,8 @@
                                                    StoreInst *Store0) {
   DEBUG(dbgs() << "can Sink? : "; Store0->dump(); dbgs() << "\n");
   BasicBlock *BB0 = Store0->getParent();
-  for (BasicBlock::reverse_iterator RBI = BB1->rbegin(), RBE = BB1->rend();
-       RBI != RBE; ++RBI) {
-    Instruction *Inst = &*RBI;
-
-    auto *Store1 = dyn_cast<StoreInst>(Inst);
+  for (Instruction &Inst : reverse(*BB1)) {
+    auto *Store1 = dyn_cast<StoreInst>(&Inst);
     if (!Store1)
       continue;