This is an archive of the discontinued LLVM Phabricator instance.

R600/SI: Use SGPR_64 as reister class for S_LOAD_DWORDX2
AbandonedPublic

Authored by arsenm on Aug 1 2014, 4:05 PM.

Details

Reviewers
None
Summary

There's no reason to ever load into the special registers
in the superclass SReg_64. This helps avoid the verifier
error "Expected a SGPR_64 register, but got a SReg_64 register"
because SGPR_64 is not a superclass of SReg_64 when trying
to use a loaded kernel argument register in some cases.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 12124.Aug 1 2014, 4:05 PM
arsenm retitled this revision from to R600/SI: Use SGPR_64 as reister class for S_LOAD_DWORDX2 .
arsenm updated this object.
arsenm edited the test plan for this revision. (Show Details)
arsenm added a subscriber: Unknown Object (MLST).
arsenm abandoned this revision.Aug 1 2014, 4:16 PM

Actually I think this will just hide a more general problem since lots of places use SReg_64 destinations