[Mlir-commits] [mlir] [mlir][memref-to-spirv]: Reverse Image Load Coordinates (PR #160495)

[Igor Wodiany]

IgWod-IMG wrote:

I am a bit confused, but maybe a picture will help. Let's look at the Figure 1 in [here](https://docs.vulkan.org/spec/latest/chapters/textures.html). Is what you are trying to say that in `memref` the first dimension would be height, and the second dimension would be width, but in Vulkan (as in the figure) the first dimension is width, whereas the second dimension is height? And since in both cases you want to access data row by row, we need to reverse indices? So basically, without this patch the conversion silently changes iteration from row-major to column-major?

Am I understanding it right?

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