This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Move R600 test compatability hack
ClosedPublic

Authored by arsenm on Feb 10 2020, 6:32 AM.

Details

Reviewers
rampitec
jvesely
Summary

Instead of handling the r600 intrinsics on amdgcn, handle the amdgcn
intrinsics on r600.

Diff Detail

Event Timeline

arsenm created this revision.Feb 10 2020, 6:32 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 10 2020, 6:32 AM
foad added a subscriber: foad.Feb 10 2020, 7:45 AM

Typo in subject compatability -> compatibility. (Sorry I don't have any comments on the actual patch.)

Is there any test left that tests the r600 intrinsics? If yes, LGTM.

Is there any test left that tests the r600 intrinsics? If yes, LGTM.

These should already be covered by test/CodeGen/AMDGPU/r600.work-item-intrinsics.ll

This revision is now accepted and ready to land.Feb 10 2020, 8:59 AM