[Mlir-commits] [mlir] [mlir][ArmSME] Remove `ConvertIllegalShapeCastOpsToTransposes` (PR #139706)

[Benjamin Maxwell]

https://github.com/MacDue commented:

> I’d prefer not to, mainly because there’s a fundamental difference between vector.transpose and vector.shape_cast. The former involves actual data movement, while the latter is not supposed to - as noted in the [docs](https://mlir.llvm.org/docs/Dialects/Vector/#vectorshape_cast-vectorshapecastop):

Not blocking this PR, but I don't really agree with this stance, `vector.shape_cast` has never really been a no-op (the docs have been outdated for a long time). It's no different from any other operation e.g. `vector.transpose`, that may mean data movement, or may fold away (e.g. into a load/store). :slightly_smiling_face: 

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