[PATCH] D74544: [MLIR] Add naive fusion of parallel loops.

Alexander Belyaev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 13 23:29:06 PST 2020


pifon2a marked an inline comment as done.
pifon2a added inline comments.


================
Comment at: mlir/lib/Transforms/ParallelLoopFusion.cpp:137
+static mlir::PassRegistration<mlir::LoopFusion>
+    pass("parallel-loop-fusion", "Fuse parallel loop nests");
----------------
mehdi_amini wrote:
> Can this pass be located under the loop dialect?
@mehdi_amini Do you mean moving this code to `mlir/lib/Transforms/LoopFusion.cpp` where fusion on affine loops is implemented? That file is quite big already. Or do you mean having it in `Dialect/LoopOps/Transforms` similarly to `Dialect/Linalg/Transforms`? If it is the latter, then there might be several other passes in `lib/Transforms` that have to be moved to respective directories.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74544





More information about the llvm-commits mailing list