This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Add GFX11 feature for subtargets with more VGPRs
ClosedPublic

Authored by foad on Sep 23 2022, 1:05 AM.

Details

Summary

The full complement of physical VGPRs for GFX11 is 50% more than GFX10.
Some subtargets have this, others stay the same as GFX10. This affects
occupancy calculations.

Diff Detail

Event Timeline

foad created this revision.Sep 23 2022, 1:05 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 23 2022, 1:05 AM
foad requested review of this revision.Sep 23 2022, 1:05 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 23 2022, 1:05 AM
arsenm added inline comments.Sep 23 2022, 8:00 AM
llvm/lib/Target/AMDGPU/AMDGPU.td
699

FullVGPRs instead of extra?

foad updated this revision to Diff 462502.Sep 23 2022, 8:31 AM

Rename feature.

foad retitled this revision from [AMDGPU] Add GFX11 feature for subtargets with extra VGPRs to [AMDGPU] Add GFX11 feature for subtargets with more VGPRs.Sep 23 2022, 8:31 AM
foad edited the summary of this revision. (Show Details)
foad marked an inline comment as done.
This revision is now accepted and ready to land.Sep 23 2022, 10:21 AM
This revision was automatically updated to reflect the committed changes.