commit 5d5134f676a178d37bcccc721cbf0c9c1c97a61c
author Sanjay Patel <spatel@rotateright.com> Fri May 13 20:24:53 2016 +0000
committer Sanjay Patel <spatel@rotateright.com> Fri May 13 20:24:53 2016 +0000
tree 3bc1172fe722a564bd26e7bb3b2d901e11c42459
parent 25a7330cc8fad30267a1a17b5aa4846a6d71f6ca

use range-loops; NFCI

llvm-svn: 269471

llvm/lib/Transforms/Utils/SimplifyCFG.cpp

diff --git a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp
index 7f5e7eb..0eebd8f 100644
--- a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp
+++ b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp
@@ -3865,12 +3865,12 @@
 
   // Gather dead cases.
   SmallVector<ConstantInt*, 8> DeadCases;
-  for (SwitchInst::CaseIt I = SI->case_begin(), E = SI->case_end(); I != E; ++I) {
-    if ((I.getCaseValue()->getValue() & KnownZero) != 0 ||
-        (I.getCaseValue()->getValue() & KnownOne) != KnownOne) {
-      DeadCases.push_back(I.getCaseValue());
+  for (auto &Case : SI->cases()) {
+    if ((Case.getCaseValue()->getValue() & KnownZero) != 0 ||
+        (Case.getCaseValue()->getValue() & KnownOne) != KnownOne) {
+      DeadCases.push_back(Case.getCaseValue());
       DEBUG(dbgs() << "SimplifyCFG: switch case '"
-                   << I.getCaseValue() << "' is dead.\n");
+                   << Case.getCaseValue() << "' is dead.\n");
     }
   }
 
@@ -3905,8 +3905,8 @@
   }
 
   // Remove dead cases from the switch.
-  for (unsigned I = 0, E = DeadCases.size(); I != E; ++I) {
-    SwitchInst::CaseIt Case = SI->findCaseValue(DeadCases[I]);
+  for (ConstantInt *DeadCase : DeadCases) {
+    SwitchInst::CaseIt Case = SI->findCaseValue(DeadCase);
     assert(Case != SI->case_default() &&
            "Case was not found. Probably mistake in DeadCases forming.");
     if (HasWeight) {