[Mlir-commits] [mlir] [mlir][Linalg] Add transform to convert linalg.copy into memref.copy (PR #132422)

Pablo Antonio Martinez llvmlistbot at llvm.org
Fri Mar 28 10:47:07 PDT 2025


pabloantoniom wrote:

> Thanks for contributing to the transform dialect, @pabloantoniom! IMO, code quality wise this is very good.
> 
> My understanding is that this runs _after_ bufferization as it only applies to linalg-on-memref. As such it shouldn't have (unfortunate) interactions with bufferization. With that in mind, my understanding is that this is a valid (though _very_ gradual) lowering step. It does raise the question for me: is there currently a pass that does this/something similar? If that's so, could you maybe mention it in the summary, i.e. commit message?
> 
> I am okay with this to go in with just my nits addressed, though if you could give it another couple days for others to have a look, that would be great. 👍

I have changed a couple of small things (mainly rewording) and added a new test. If you like the changes and nobody else has any objection I will merge this shortly. Thanks Rolf! 👍 

Re your question: I'm not aware of something like this.

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


More information about the Mlir-commits mailing list