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

Changpeng Fang via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 9 14:11:32 PDT 2016


cfang updated this revision to Diff 67415.
cfang added a comment.

Update LIT tests based on new intrinsics definition for image load and image:

1. image intrinsics name has three suffixes for types of vdata, address and resource, respectively. Something like: declare void @llvm.amdgcn.image.store.v4f32.v2i32.v8i32(<4 x float>, <2 x i32>, <8 x i32>, i32, i1, i1, i1, i1) #0 declare <4 x float> @llvm.amdgcn.image.load.v4f32.v4i32.v8i32(<4 x i32>, <8 x i32>, i32, i1, i1, i1, i1) #1

2. The four flags exposed are in the following order: glc, slc, lwe and da

3. unrm is not exposed and default to 1


https://reviews.llvm.org/D23286

Files:
  include/llvm/IR/IntrinsicsAMDGPU.td
  lib/Target/AMDGPU/SIInstructions.td
  test/CodeGen/AMDGPU/llvm.amdgcn.image.ll
  test/CodeGen/AMDGPU/llvm.amdgcn.s.waitcnt.ll
  test/CodeGen/AMDGPU/wqm.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23286.67415.patch
Type: text/x-patch
Size: 16822 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160809/400fe245/attachment.bin>


More information about the llvm-commits mailing list