[Mlir-commits] [mlir] [MLIR][Linalg] Remove matmul_transpose variants (PR #147961)

[Renato Golin]

rengolin wrote:

I see a lot of usage of matmul transpose in IREE: https://github.com/search?q=repo%3Airee-org%2Firee%20matmul_transpose&type=code

A few things I can see from just that search:
* Some usages are "representational", ie. could easily be replaced with the alternative.
* `if (!isa<linalg::MatmulTransposeBOp, linalg::BatchMatmulTransposeBOp>(op)) {` could be replaced with the matcher that @qed discusses above. Here, just check something like `isMapTranspose` that works on both `matmul` and `batch_matmul`.
* I assume `{lowering_config = ...}` attribute could also be applied to `matmul`
* Upstream tests show that transforms/passes still work on the alternative, so IREE should benefit from those.
* IREE specific passes may need to adapt, but if made on par with upstream, may also work out of the box.

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