[Mlir-commits] [mlir] [mlir] Canonicalize tensor.extract_slice (linalg.fill) (PR #112619)

[llvmlistbot at llvm.org]

https://github.com/MaheshRavishankar requested changes to this pull request.

I think you should avoid using a `tensor.empty` for the destination of the `fill` op. It is better just using a slice of a destination.

Come to think of it, that way you could drop the multiple use constraint too..

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