[PATCH] D73145: [mlir][Linalg] Introduce folding patterns to remove certain MemRefCastOp

Nicolas Vasilache via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 28 12:23:25 PST 2020


nicolasvasilache marked 5 inline comments as done.
nicolasvasilache added inline comments.


================
Comment at: mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp:78
+  // TODO(ntv, rriddle): this may be a good use case for an OpInterface.
+  for (auto &u : castOp.getResult().getUses()) {
+    if (isa<LinalgOp>(u.getOwner()) || isa<SliceOp>(u.getOwner()))
----------------
rriddle wrote:
> nit: getUsers
incorrectly carried over, case removed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73145





More information about the llvm-commits mailing list