This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Add handling of 160 bit registers in analyzeResourceUsage
ClosedPublic

Authored by rampitec on Nov 6 2019, 12:41 PM.

Details

Summary

This was omitted. Also SReg_96Reg missed IsSGPR assignment.

Diff Detail

Event Timeline

rampitec created this revision.Nov 6 2019, 12:41 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 6 2019, 12:41 PM
arsenm added a comment.Nov 6 2019, 2:40 PM

Needs testcase

Needs testcase

Do we have an existing testcase for this part?

arsenm added a comment.Nov 6 2019, 3:13 PM

Needs testcase

Do we have an existing testcase for this part?

101abd219b3dc77fb531e569e4b6bd86ed48b5c1 fixed the same problem for SGPR96 and tested this with inline asm

rampitec updated this revision to Diff 228160.Nov 6 2019, 3:34 PM

Added test.

arsenm accepted this revision.Nov 6 2019, 3:37 PM

LGTM

This revision is now accepted and ready to land.Nov 6 2019, 3:37 PM
This revision was automatically updated to reflect the committed changes.