[PATCH] D77486: [MLIR][NFC] fix name operand -> op

[Uday Bondhugula via Phabricator via llvm-commits]

bondhugula added inline comments.


================
Comment at: mlir/lib/Transforms/Utils/GreedyPatternRewriteDriver.cpp:184
         for (auto result : op->getResults())
-          for (auto *operand : result.getUsers())
-            addToWorklist(operand);
+          for (auto *op : result.getUsers())
+            addToWorklist(op);
----------------
stephenneuendorffer wrote:
> This seems like a bad idea to me, since you're shadowing another outer op.
I hadn't even noticed - thanks! Fixed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77486