[llvm] r300350 - Use range-for in a few places

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 14 10:42:13 PDT 2017


Author: sanjoy
Date: Fri Apr 14 12:42:12 2017
New Revision: 300350

URL: http://llvm.org/viewvc/llvm-project?rev=300350&view=rev
Log:
Use range-for in a few places

Modified:
    llvm/trunk/include/llvm/Analysis/ScalarEvolutionExpressions.h

Modified: llvm/trunk/include/llvm/Analysis/ScalarEvolutionExpressions.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Analysis/ScalarEvolutionExpressions.h?rev=300350&r1=300349&r2=300350&view=diff
==============================================================================
--- llvm/trunk/include/llvm/Analysis/ScalarEvolutionExpressions.h (original)
+++ llvm/trunk/include/llvm/Analysis/ScalarEvolutionExpressions.h Fri Apr 14 12:42:12 2017
@@ -617,9 +617,9 @@ namespace llvm {
     const SCEV *visitAddExpr(const SCEVAddExpr *Expr) {
       SmallVector<const SCEV *, 2> Operands;
       bool Changed = false;
-      for (int i = 0, e = Expr->getNumOperands(); i < e; ++i) {
-        Operands.push_back(((SC*)this)->visit(Expr->getOperand(i)));
-        Changed |= Expr->getOperand(i) != Operands.back();
+      for (auto *Op : Expr->operands()) {
+        Operands.push_back(((SC*)this)->visit(Op));
+        Changed |= Op != Operands.back();
       }
       return !Changed ? Expr : SE.getAddExpr(Operands);
     }
@@ -627,9 +627,9 @@ namespace llvm {
     const SCEV *visitMulExpr(const SCEVMulExpr *Expr) {
       SmallVector<const SCEV *, 2> Operands;
       bool Changed = false;
-      for (int i = 0, e = Expr->getNumOperands(); i < e; ++i) {
-        Operands.push_back(((SC*)this)->visit(Expr->getOperand(i)));
-        Changed |= Expr->getOperand(i) != Operands.back();
+      for (auto *Op : Expr->operands()) {
+        Operands.push_back(((SC*)this)->visit(Op));
+        Changed |= Op != Operands.back();
       }
       return !Changed ? Expr : SE.getMulExpr(Operands);
     }
@@ -644,9 +644,9 @@ namespace llvm {
     const SCEV *visitAddRecExpr(const SCEVAddRecExpr *Expr) {
       SmallVector<const SCEV *, 2> Operands;
       bool Changed = false;
-      for (int i = 0, e = Expr->getNumOperands(); i < e; ++i) {
-        Operands.push_back(((SC*)this)->visit(Expr->getOperand(i)));
-        Changed |= Expr->getOperand(i) != Operands.back();
+      for (auto *Op : Expr->operands()) {
+        Operands.push_back(((SC*)this)->visit(Op));
+        Changed |= Op != Operands.back();
       }
       return !Changed ? Expr
                       : SE.getAddRecExpr(Operands, Expr->getLoop(),
@@ -656,9 +656,9 @@ namespace llvm {
     const SCEV *visitSMaxExpr(const SCEVSMaxExpr *Expr) {
       SmallVector<const SCEV *, 2> Operands;
       bool Changed = false;
-      for (int i = 0, e = Expr->getNumOperands(); i < e; ++i) {
-        Operands.push_back(((SC *)this)->visit(Expr->getOperand(i)));
-        Changed |= Expr->getOperand(i) != Operands.back();
+      for (auto *Op : Expr->operands()) {
+        Operands.push_back(((SC *)this)->visit(Op));
+        Changed |= Op != Operands.back();
       }
       return !Changed ? Expr : SE.getSMaxExpr(Operands);
     }
@@ -666,9 +666,9 @@ namespace llvm {
     const SCEV *visitUMaxExpr(const SCEVUMaxExpr *Expr) {
       SmallVector<const SCEV *, 2> Operands;
       bool Changed = false;
-      for (int i = 0, e = Expr->getNumOperands(); i < e; ++i) {
-        Operands.push_back(((SC*)this)->visit(Expr->getOperand(i)));
-        Changed |= Expr->getOperand(i) != Operands.back();
+      for (auto *Op : Expr->operands()) {
+        Operands.push_back(((SC*)this)->visit(Op));
+        Changed |= Op != Operands.back();
       }
       return !Changed ? Expr : SE.getUMaxExpr(Operands);
     }




More information about the llvm-commits mailing list