[PATCH] D58957: [AMDGPU] Add an experimental buffer fat pointer address space.

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: docs/AMDGPUUsage.rst:299-300
+unsupported in the backend. It exposes a non-integral pointer that is in future
+intended to support the modelling of 128-bit buffer descriptors + a 32-bit
+offset into the buffer descriptor (in total encapsulating a 160-bit 'pointer'),
+allowing us to use normal LLVM load/store/atomic operations to model the buffer
----------------
I don't understand why you would blend these. You just need the 128-bit pointer, and then the intrinsic accessing it will have a 32-bit offset operand that isn't part of the pointer


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58957/new/

https://reviews.llvm.org/D58957