commit 241223d5a9bdb6f36bd0763898b029b22d1f0395
author Adrian Kuegel <akuegel@google.com> Fri Dec 03 13:07:54 2021 +0100
committer Copybara-Service <copybara-worker@google.com> Fri Dec 03 04:10:43 2021 -0800
tree 6848c425b2879315a8df7bf6bdf9d16a2d956a52
parent eb3cbc26be083fd637df5667c654933955005d7b

[mlir][NFC] Use const reference for loop variables.

GitOrigin-RevId: 04d083b19e6eb80587cc27fa3996121c8c4bddbc

lib/Dialect/Linalg/ComprehensiveBufferize/BufferizableOpInterface.cpp

diff --git a/lib/Dialect/Linalg/ComprehensiveBufferize/BufferizableOpInterface.cpp b/lib/Dialect/Linalg/ComprehensiveBufferize/BufferizableOpInterface.cpp
index 61c4f8e..6a75398 100644
--- a/lib/Dialect/Linalg/ComprehensiveBufferize/BufferizableOpInterface.cpp
+++ b/lib/Dialect/Linalg/ComprehensiveBufferize/BufferizableOpInterface.cpp
@@ -556,14 +556,14 @@
       reifiedShapes = true;
       OpResult resultValue = shapedValue.dyn_cast<OpResult>();
       auto &shape = resultDims[resultValue.getResultNumber()];
-      for (auto dim : enumerate(allocMemRefType.getShape()))
+      for (const auto &dim : enumerate(allocMemRefType.getShape()))
         if (ShapedType::isDynamic(dim.value()))
           dynShape.push_back(shape[dim.index()]);
     }
   }
 
   if (!reifiedShapes) {
-    for (auto dim : enumerate(allocMemRefType.getShape()))
+    for (const auto &dim : enumerate(allocMemRefType.getShape()))
       if (ShapedType::isDynamic(dim.value())) {
         assert((shapedValue.getType().isa<UnrankedMemRefType>() ||
                 shapedValue.getType().isa<MemRefType>()) &&

lib/Dialect/Linalg/ComprehensiveBufferize/ModuleBufferization.cpp

diff --git a/lib/Dialect/Linalg/ComprehensiveBufferize/ModuleBufferization.cpp b/lib/Dialect/Linalg/ComprehensiveBufferize/ModuleBufferization.cpp
index 9ed8055..d24e7e8 100644
--- a/lib/Dialect/Linalg/ComprehensiveBufferize/ModuleBufferization.cpp
+++ b/lib/Dialect/Linalg/ComprehensiveBufferize/ModuleBufferization.cpp
@@ -370,7 +370,7 @@
     SmallVector<Type> argumentTypes;
     // Iterate on each function argument and check it it was marked with a
     // desired layout.
-    for (auto it : llvm::enumerate(funcOp.getType().getInputs())) {
+    for (const auto &it : llvm::enumerate(funcOp.getType().getInputs())) {
       int argNumber = it.index();
       Type inputType = it.value();
       auto memrefType = inputType.dyn_cast<MemRefType>();