This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Add Clang builtin intrinsics for ds_swizzle
ClosedPublic

Authored by cfang on Aug 18 2016, 11:11 AM.

Details

Diff Detail

Repository
rL LLVM

Event Timeline

cfang updated this revision to Diff 68574.Aug 18 2016, 11:11 AM
cfang retitled this revision from to AMDGPU: Add Clang builtin intrinsics for ds_swizzle.
cfang updated this object.
cfang added reviewers: wdng, arsenm, tstellarAMD.
cfang added subscribers: arsenm, llvm-commits.
arsenm added inline comments.Aug 18 2016, 11:18 AM
include/clang/Basic/BuiltinsAMDGPU.def
79 ↗(On Diff #68574)

The constant operand should marked with I (and there should be a test that it errors with a non-constant)

cfang updated this revision to Diff 68604.Aug 18 2016, 2:13 PM

Update based on Matt's comments:

  1. Mark constant integer argument type as "Ii"
  2. add error test when argument is not constant integer.
arsenm accepted this revision.Aug 18 2016, 2:19 PM
arsenm edited edge metadata.

LGTM

This revision is now accepted and ready to land.Aug 18 2016, 2:19 PM
This revision was automatically updated to reflect the committed changes.