This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Pre-checkin updated lit tests for D123525.
ClosedPublic

Authored by hsmhsm on Apr 14 2022, 11:39 AM.

Details

Summary

Fix indentation within the lit test - agpr-copy-no-free-registers.ll.

Diff Detail

Event Timeline

hsmhsm created this revision.Apr 14 2022, 11:39 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 14 2022, 11:39 AM
Herald added subscribers: kerbowa, t-tye, tpr and 5 others. · View Herald Transcript
hsmhsm requested review of this revision.Apr 14 2022, 11:39 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 14 2022, 11:39 AM
hsmhsm edited the summary of this revision. (Show Details)Apr 14 2022, 11:41 AM
hsmhsm edited the summary of this revision. (Show Details)
rampitec added inline comments.Apr 14 2022, 1:57 PM
llvm/test/CodeGen/AMDGPU/agpr-usage-should-fail-on-gfx900.ll
1 ↗(On Diff #422930)

I do not see why do you need to split it and for that matter this test is too complicated for a simple check that 'a' constraint is not accepted. It was OK as a side check, but not as a separate test.

llvm/test/CodeGen/AMDGPU/spill-agpr-gfx908.ll
1 ↗(On Diff #422930)

Again, why split it? It is the same test. If you need an extra function with 6 vgprs just add it.

hsmhsm updated this revision to Diff 423009.Apr 14 2022, 8:49 PM

Restore spill-vgpr-to-agpr.ll and spill-agpr.ll.

hsmhsm edited the summary of this revision. (Show Details)Apr 14 2022, 8:50 PM
hsmhsm marked 2 inline comments as done.Apr 14 2022, 9:02 PM
rampitec accepted this revision.Apr 15 2022, 10:39 AM
This revision is now accepted and ready to land.Apr 15 2022, 10:39 AM
This revision was landed with ongoing or failed builds.Apr 16 2022, 10:16 PM
This revision was automatically updated to reflect the committed changes.