[Mlir-commits] [mlir] [mlir][memref]: Add expand/collapse rewrite pattern to MemRef::CopyOp (PR #67808)

Nicolas Vasilache llvmlistbot at llvm.org
Mon Oct 2 04:18:31 PDT 2023


nicolasvasilache wrote:

Hmm, could you please elaborate on the rationale for this?

I am generally quite opposed to adding transformations on `memref.copy` because this is a very specific specialization of `linalg.copy` that almost always results in code duplication.

Have you considered using a `linalg.copy` instead and using/extending the existing transforms on such abstractions?

If anything, I would just delete `memref.copy`.



https://github.com/llvm/llvm-project/pull/67808


More information about the Mlir-commits mailing list