[PATCH] D65097: AMDGPU: Add offsets to MMO when lowering buffer intrinsics

Tom Stellard via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 24 16:15:18 PDT 2019


tstellar marked an inline comment as done.
tstellar added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:6841-6842
 
   case Intrinsic::amdgcn_raw_buffer_store:
   case Intrinsic::amdgcn_raw_buffer_store_format: {
     SDValue VData = Op.getOperand(2);
----------------
arsenm wrote:
> Why aren't these handled?
I didn't handle the raw or struct intrinsics, because I wasn't sure if the swizzling or something else from the resource descriptor affected the offset value.  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65097





More information about the llvm-commits mailing list