[Mlir-commits] [mlir] [mlir][vector][memref] Add `alignment` attribute to memory access ops (PR #144344)

[Jakub Kuderski]

kuhar wrote:

> What high-level information does `memref<?xi8>` have? It's an array of bytes...
> why we should keep abusing memrefs when a pointer is actually needed

Going back to the example of merging allocations, we want to do that at the level of memref common for both llvm and spirv. At that point you do have the original memref types and can set the alignment information. Doing the same transformation at the level of spirv and close to llvm (with `ptr.ptr`) seems too late for me, and there's no better place to store alignment info then on load/store ops.

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