This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Add switch to enable architected SGPRs.
ClosedPublic

Authored by cdevadas on Feb 10 2023, 12:51 AM.

Diff Detail

Event Timeline

cdevadas created this revision.Feb 10 2023, 12:51 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 10 2023, 12:51 AM
cdevadas requested review of this revision.Feb 10 2023, 12:51 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 10 2023, 12:51 AM
foad accepted this revision.Feb 10 2023, 1:38 AM

LGTM.

llvm/lib/Target/AMDGPU/AMDGPU.td
872

Nit: I'd prefer a better description, but I don't have a suggestion. I've never really understood what "architected" is supposed to mean.

This revision is now accepted and ready to land.Feb 10 2023, 1:38 AM
rampitec accepted this revision.Feb 10 2023, 10:56 AM
This revision was landed with ongoing or failed builds.Feb 13 2023, 8:41 AM
This revision was automatically updated to reflect the committed changes.