This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Add num spilled s/vgprs to metadata
ClosedPublic

Authored by kzhuravl on Nov 21 2017, 1:48 PM.

Diff Detail

Event Timeline

kzhuravl created this revision.Nov 21 2017, 1:48 PM
t-tye added inline comments.Nov 21 2017, 5:01 PM
docs/AMDGPUUsage.rst
1298–1300

Suggest clarifying to:

Number of stores from a scalar register to a register allocator created spill location.

1301–1303

Suggest clarifying to:

Number of stores from a vector register to a register allocator created spill location.

artem.tamazov accepted this revision.Nov 22 2017, 4:39 AM
This revision is now accepted and ready to land.Nov 22 2017, 4:39 AM
kzhuravl updated this revision to Diff 124443.Nov 27 2017, 12:37 PM
kzhuravl marked 2 inline comments as done.

Address review feedback.

t-tye accepted this revision.Nov 27 2017, 9:15 PM

LGTM provided it meets the needs of the tools? Have they reviewed?

LGTM provided it meets the needs of the tools? Have they reviewed?

Yes.

This revision was automatically updated to reflect the committed changes.