This is an archive of the discontinued LLVM Phabricator instance.

Clean up AMDGCN tests
ClosedPublic

Authored by yaxunl on Feb 15 2018, 9:45 AM.

Details

Diff Detail

Repository
rC Clang

Event Timeline

yaxunl created this revision.Feb 15 2018, 9:45 AM
t-tye accepted this revision.Feb 15 2018, 10:12 AM

LGTM Plus I think there was a test with giz in its name. Should that be renamed?

This revision is now accepted and ready to land.Feb 15 2018, 10:12 AM

LGTM Plus I think there was a test with giz in its name. Should that be renamed?

That's test/CodeGenOpenCL/amdgpu-env-amdgiz.cl. I already renamed it in my last commit.

b-sumner accepted this revision.Feb 15 2018, 11:01 AM

Looks good to me.

This revision was automatically updated to reflect the committed changes.