[PATCH] D74342: [mlir] Add MemRef filter to affine data copy optimization

Thu Feb 13 21:29:22 PST 2020

bondhugula accepted this revision.
bondhugula added a comment.

In D74342#1872854 <https://reviews.llvm.org/D74342#1872854>, @dcaballe wrote:

> Rebasing on top of latest mlir-opt and TestLoopFusion changes.
>  If no more comments, I'll address 'gatherLoops' concerns in a
>  separate commit and keep this as a simple rafactoring of that
>  utility.

Yes, gatherLoops can be addressed separately; if better, you could keep gatherLoops private (and duplicated), and it could be addressed when made public.  Either way is fine for me.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74342