[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