This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Precommit test for D141576
ClosedPublic

Authored by Pierre-vh on Jan 17 2023, 12:50 AM.

Diff Detail

Event Timeline

Pierre-vh created this revision.Jan 17 2023, 12:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 17 2023, 12:50 AM
Pierre-vh requested review of this revision.Jan 17 2023, 12:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 17 2023, 12:50 AM
arsenm accepted this revision.Jan 17 2023, 8:34 AM
This revision is now accepted and ready to land.Jan 17 2023, 8:34 AM

An identical case with SGPRs and AGPRs wouldn't hurt too

An identical case with SGPRs and AGPRs wouldn't hurt too

Replacing the vregs with agpr/sgpr doesn't seem to work, I think the test case needs to have more changes to recreate the same issue with something other than vregs.
I'll commit this now as the fix is blocking a ticket, but if you know how to make the test work with s/agprs I can do a separate commit to increase testing coverage

This revision was landed with ongoing or failed builds.Jan 18 2023, 12:49 AM
This revision was automatically updated to reflect the committed changes.