[PATCH] D72210: [NFC] Fixes -Wrange-loop-analysis warnings

[Mark de Wever via Phabricator via llvm-commits]

Mordante updated this revision to Diff 236239.
Mordante added a comment.

Rebased on master. This adds one new fix and removes the ones done by others.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72210/new/

https://reviews.llvm.org/D72210

Files:
  mlir/lib/Dialect/Linalg/Utils/Utils.cpp
  polly/lib/Analysis/ScopGraphPrinter.cpp


Index: polly/lib/Analysis/ScopGraphPrinter.cpp
===================================================================
--- polly/lib/Analysis/ScopGraphPrinter.cpp
+++ polly/lib/Analysis/ScopGraphPrinter.cpp
@@ -178,7 +178,7 @@
 
     RegionInfo *RI = R->getRegionInfo();
 
-    for (const auto &BB : R->blocks())
+    for (auto *BB : R->blocks())
       if (RI->getRegionFor(BB) == R)
         O.indent(2 * (depth + 1))
             << "Node"
Index: mlir/lib/Dialect/Linalg/Utils/Utils.cpp
===================================================================
--- mlir/lib/Dialect/Linalg/Utils/Utils.cpp
+++ mlir/lib/Dialect/Linalg/Utils/Utils.cpp
@@ -114,7 +114,7 @@
   SmallVector<ValueHandle, 4> lbs;
   SmallVector<ValueHandle, 4> ubs;
   SmallVector<int64_t, 4> steps;
-  for (const Value range : ranges) {
+  for (Value range : ranges) {
     assert(range.getType() && "expected linalg.range type");
     assert(range.getDefiningOp() && "need operations to extract range parts");
     RangeOp rangeOp = cast<RangeOp>(range.getDefiningOp());


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D72210.236239.patch
Type: text/x-patch
Size: 1041 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200105/2de1c61a/attachment.bin>