[Mlir-commits] [mlir] [mlir][linalg] Extract `GeneralizePadOpPattern` into a standalone transformation (PR #117329)

Renato Golin llvmlistbot at llvm.org
Fri Nov 22 07:58:08 PST 2024


rengolin wrote:

The reasoning here looks good to me, haven't looked at the PR yet.

Side comment: this is another "layering violation", where `tensor.pad` "lowers" to `linalg.fill`. Tensor dialect should not know about linalg at all. Yet another candidate to move to linalg. (https://discourse.llvm.org/t/rfc-move-tensor-pack-and-tensor-unpack-into-linalg/83096/)

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


More information about the Mlir-commits mailing list