SCEV: Use range-based for loop and fold variable into assert.

llvm-svn: 208476

1 files changed, 4 insertions, 6 deletions

diff --git a/llvm/lib/Analysis/ScalarEvolution.cpp b/llvm/lib/Analysis/ScalarEvolution.cpp
index f065d851a00..5f06e8238c9 100644
--- a/llvm/lib/Analysis/ScalarEvolution.cpp
+++ b/llvm/lib/Analysis/ScalarEvolution.cpp
@@ -7214,14 +7214,12 @@ static void findArrayDimensionsRec(ScalarEvolution &SE,
     return;
   }
 
-  const SCEV *Zero = SE.getConstant(GCD->getType(), 0);
-
-  for (unsigned I = 0; I < Terms.size(); ++I) {
+  for (const SCEV *&Term : Terms) {
     // Normalize the terms before the next call to findArrayDimensionsRec.
     const SCEV *Q, *R;
-    SCEVDivision::divide(SE, Terms[I], GCD, &Q, &R);
-    assert(R == Zero && "GCD does not evenly divide one of the terms");
-    Terms[I] = Q;
+    SCEVDivision::divide(SE, Term, GCD, &Q, &R);
+    assert(R->isZero() && "GCD does not evenly divide one of the terms");
+    Term = Q;
   }
 
   // Remove all SCEVConstants.