[Mlir-commits] [mlir] [mlir][vector] Add patterns for vector masked load/store (PR #74834)

[Jakub Kuderski]

kuhar wrote:

> Hi @kuhar,
> 
> I rewrite my patch to convert vector.maskedload to vector.load and convert vector.maskedstore to memref.store. Does it make sense for these patterns? Thanks for your help and review.

I left some comments. This lowering looks like the right direction to me, but I think we need to introduce control flow to avoid out-of-bounds memory accesses (instead of selects).

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