[PATCH] D17614: AMDGPU/SI: Implement DS_PERMUTE/DS_BPERMUTE Instruction Definitions and Intrinsics.

Changpeng Fang via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 26 09:25:34 PST 2016


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

Update the patch based on Matt's Review:

1. Check only Offset0Imm and move the check one line ahead.
2. It is safe to remove M0 from the Uses list for ds_permute.ds_bpermute.
3. split the LIT test for ds_permute and ds_bpermute separately.


http://reviews.llvm.org/D17614

Files:
  include/llvm/IR/IntrinsicsAMDGPU.td
  lib/Target/AMDGPU/SIInstrInfo.cpp
  lib/Target/AMDGPU/SIInstrInfo.td
  lib/Target/AMDGPU/VIInstructions.td
  test/CodeGen/AMDGPU/llvm.amdgcn.ds.bpermute.ll
  test/CodeGen/AMDGPU/llvm.amdgcn.ds.permute.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17614.49202.patch
Type: text/x-patch
Size: 4368 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160226/c73a5b13/attachment.bin>


More information about the llvm-commits mailing list