[PATCH] D73165: [MLIR][Linalg] Allow fusion of more than 2 linalg ops.

Nicolas Vasilache via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 24 11:48:50 PST 2020


nicolasvasilache added a comment.

please make the commit message significantly more descriptive: what was the problem, how did you fix it etc



================
Comment at: mlir/lib/Dialect/Linalg/Transforms/Fusion.cpp:345
+        auto originalOpInLinalgOpsVector =
+            std::find_if(linalgOps.begin(), linalgOps.end(),
+                         [&](const Operation *op) { return op == originalOp; });
----------------
rriddle wrote:
> llvm::find(linalgOps, originalOp)?
does @rriddle's solution work?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73165





More information about the llvm-commits mailing list