[PATCH] D23286: AMDGPU/SI: Propose to redefine image load/store intrinsics

[Changpeng Fang via llvm-commits]

cfang added a comment.



> Image loads/stores are a different category. There are several categories but you can split all image opcodes into two:

> 

> - takes a floating-point address (all sample and gather opcodes) and also a sampler descriptor, the "unorm" and "da" bits apply here

> - takes an integer address (all load, store and atomic opcodes), the "unorm" bit is irrelevant (or must be 1 in some cases) and only the "da" bit applies


Thanks, This is what we are doing currently.
This patch handle image load/store;  unorm ==1 and expose da

Another patch (https://reviews.llvm.org/D22838) handle sampler related image intrinsics which expose unorm and da


https://reviews.llvm.org/D23286