[Mlir-commits] [mlir] [mlir][Linalg] Add transform to convert linalg.copy into memref.copy (PR #132422)
Rolf Morel
llvmlistbot at llvm.org
Sun Mar 23 11:21:03 PDT 2025
https://github.com/rolfmorel approved this pull request.
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. :+1:
https://github.com/llvm/llvm-project/pull/132422
More information about the Mlir-commits
mailing list