This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Use SGPR_128 instead of SReg_128 for vregs
ClosedPublic

Authored by arsenm on Oct 9 2019, 4:36 PM.

Details

Reviewers
rampitec
Summary

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.

Diff Detail

Event Timeline

arsenm created this revision.Oct 9 2019, 4:36 PM
rampitec accepted this revision.Oct 9 2019, 4:42 PM

LGTM

This revision is now accepted and ready to land.Oct 9 2019, 4:42 PM
arsenm closed this revision.Oct 10 2019, 12:09 AM

r374284