This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Rename intrinsics to use amdgcn prefix
ClosedPublic

Authored by arsenm on Jan 21 2016, 8:08 PM.

Details

Reviewers
tstellarAMD
Summary

The intrinsic target prefix should match the target name
as it appears in the triple.

This is not yet complete, but gets most of the important ones.
llvm.AMDGPU.* intrinsics used by mesa and libclc are still handled
for compatability for now.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 45644.Jan 21 2016, 8:08 PM
arsenm retitled this revision from to AMDGPU: Rename intrinsics to use amdgcn prefix.
arsenm updated this object.
arsenm added a reviewer: tstellarAMD.
arsenm added a subscriber: llvm-commits.
tstellarAMD accepted this revision.Jan 22 2016, 7:34 AM
tstellarAMD edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Jan 22 2016, 7:34 AM
arsenm closed this revision.Jan 22 2016, 1:35 PM

r258557

test/Transforms/InstCombine/amdgcn-intrinsics.ll