[PATCH] D78423: [MLIR] NFC affine for op tiling cleanup / utility rename

[Uday Bondhugula via Phabricator via llvm-commits]

bondhugula added inline comments.


================
Comment at: mlir/lib/Dialect/Affine/Transforms/LoopTiling.cpp:98
   for (unsigned i = 0; i < width; i++) {
-    auto lbOperands = origLoops[i].getLowerBoundOperands();
-    auto ubOperands = origLoops[i].getUpperBoundOperands();
-    SmallVector<Value, 4> newLbOperands(lbOperands);
-    SmallVector<Value, 4> newUbOperands(ubOperands);
+    newLbOperands = origLoops[i].getLowerBoundOperands();
+    newUbOperands = origLoops[i].getUpperBoundOperands();
----------------
rriddle wrote:
> Can you just use OperandRange instead?
Thanks for catching this. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78423