This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Fix some tests that did not specify -mcpu
ClosedPublic

Authored by tpr on Feb 14 2020, 1:44 AM.

Details

Summary

This fixes some tests that did not specify -mcpu. Doing that disables
all subtarget features, which gives behavior that (a) does not
necessarily correspond to any actual target, and (b) can change as we
add new subtarget features.

Change-Id: I8c0fe4fa03e9a93ef8bb722cd42d22e064526309

Diff Detail

Event Timeline

tpr created this revision.Feb 14 2020, 1:44 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 14 2020, 1:44 AM
tpr updated this revision to Diff 244599.Feb 14 2020, 2:28 AM

V2: Also added gfx1010 to memtime test.

This revision is now accepted and ready to land.Feb 14 2020, 11:57 AM
This revision was automatically updated to reflect the committed changes.
tpr added a comment.Feb 17 2020, 6:09 AM

This got landed in a slightly different form because of conflicts with other sdiv64 etc tests.

llvm/test/CodeGen/AMDGPU/sdiv64.ll