[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