This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Use SGPR instead of SReg classes
ClosedPublic

Authored by foad on Apr 22 2020, 3:12 AM.

Details

Summary

12994a70cf7 did this for 128-bit classes:

SGPR_128 only includes the real allocatable SGPRs, and SReg_128 adds
the additional non-allocatable TTMP registers. There's no point in
allocating SReg_128 vregs. This shrinks the size of the classes
regalloc needs to consider, which is usually good.

This patch extends it to all classes > 64 bits, for consistency.

Diff Detail

Event Timeline

foad created this revision.Apr 22 2020, 3:12 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 22 2020, 3:12 AM
arsenm accepted this revision.Apr 22 2020, 7:11 AM
This revision is now accepted and ready to land.Apr 22 2020, 7:11 AM
This revision was automatically updated to reflect the committed changes.