[Mlir-commits] [mlir] [mlir][vector] Refine vector.transfer_read hoisting/forwarding (PR #65770)

[Andrzej Warzyński]

banach-space wrote:

> This fixes hoisting for `collapse_shape`, but it is still broken for other ops that introduce aliasing (e.g., `expand_shape`, `arith.select`, ...). We should query an alias analysis in `noAliasingUseInLoop` instead of hard-coding ops. Still, better than before so I'd say let's land it if it's blocking.

Thanks! Agreed that a more generic would be better, but right now this is causing our convs to be broken - I really appreciate you being flexible here 🙏🏻 .

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