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

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 24 10:58:40 PDT 2019


arsenm requested changes to this revision.
arsenm added inline comments.
This revision now requires changes to proceed.


================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:6151-6152
   }
   case Intrinsic::amdgcn_raw_buffer_load:
   case Intrinsic::amdgcn_raw_buffer_load_format: {
     auto Offsets = splitBufferOffsets(Op.getOperand(3), DAG);
----------------
Why aren't these handled?


================
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);
----------------
Why aren't these handled?


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