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

River Riddle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 5 00:30:37 PDT 2020


rriddle accepted this revision.
rriddle added inline comments.


================
Comment at: mlir/lib/Transforms/Utils/GreedyPatternRewriteDriver.cpp:183
         // to revisit them.
         for (auto result : op->getResults())
+          for (auto *userOp : result.getUsers())
----------------
Can you change this to just `op->getUsers()`? It is cleaner and more efficient.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77486





More information about the llvm-commits mailing list