[PATCH] D34495: AMDGPU: Partially fix implicit.buffer.ptr intrinsic handling

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 21 19:50:59 PDT 2017


arsenm created this revision.
Herald added subscribers: t-tye, tpr, dstuttard, yaxunl, nhaehnle, wdng, kzhuravl.

This should not be treated as a different version of
private_segment_buffer. These are distinct things with
different uses and register classes, and requires the
function argument info to have more context about the
function's type and environment.

      

Also add missing test coverage for the intrinsic, and
emit an error for HSA. This also encovers that the intrinsic
is broken unless there happen to be stack objects.


https://reviews.llvm.org/D34495

Files:
  lib/Target/AMDGPU/SIFrameLowering.cpp
  lib/Target/AMDGPU/SIISelLowering.cpp
  lib/Target/AMDGPU/SIMachineFunctionInfo.cpp
  lib/Target/AMDGPU/SIMachineFunctionInfo.h
  lib/Target/AMDGPU/SIRegisterInfo.cpp
  lib/Target/AMDGPU/SIRegisterInfo.h
  test/CodeGen/AMDGPU/llvm.amdgcn.implicit.buffer.ptr.hsa.ll
  test/CodeGen/AMDGPU/llvm.amdgcn.implicit.buffer.ptr.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D34495.103514.patch
Type: text/x-patch
Size: 10969 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170622/6a5dc200/attachment.bin>


More information about the llvm-commits mailing list