[Mlir-commits] [mlir] [MLIR][XeGPU][VectorToXeGPU] Lower vector.load/store/transfer_read/transfer_write to new offsets syntax (PR #162095)

[Adam Siemieniuk]

adam-smnk wrote:

Looking at the older `update_nd_offset` semantics, the update is also restricted to the descriptor's rank.
I'd expect the new offset syntax to essentially fold `update_nd_offset` into `load_nd` while leaving descriptor creation as is.
If you want to fully avoid offsets at `create_nd_tdesc`, then an extra `memref.subview` could be added to create initial tile pointer.

Is there any clear benefit in relaxing load offset semantics? 

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