This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Resolve pseudo registers at encoding uses
ClosedPublic

Authored by Joe_Nash on Nov 3 2020, 2:45 PM.

Details

Summary

Pseudo-registers allow different register encodings
between gpu generations. Make sure we resolve the
pseudo regs to real regs whenever we get their
hardware encoding.
Using the correct encodings revealed a register
bank conflict and an unnecessary write dependency.
Tests have been updated to match.

Diff Detail

Event Timeline

Joe_Nash created this revision.Nov 3 2020, 2:45 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 3 2020, 2:45 PM
Joe_Nash requested review of this revision.Nov 3 2020, 2:45 PM
rampitec accepted this revision.Nov 3 2020, 2:48 PM

LGTM

This revision is now accepted and ready to land.Nov 3 2020, 2:48 PM
Joe_Nash updated this revision to Diff 302867.Nov 4 2020, 9:13 AM

whitespace change for lint and rerun build

This revision was landed with ongoing or failed builds.Nov 4 2020, 9:56 AM
This revision was automatically updated to reflect the committed changes.