This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Set SubRegIndex size and offset
ClosedPublic

Authored by arsenm on Jul 29 2015, 11:30 PM.

Details

Reviewers
tstellarAMD
Summary

I'm not sure what reasons the comment here could have had for not setting these. Without these set, there is an assertion hit during DWARF emission.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 30995.Jul 29 2015, 11:30 PM
arsenm retitled this revision from to AMDGPU: Set SubRegIndex size and offset.
arsenm updated this object.
arsenm added a reviewer: tstellarAMD.
arsenm added a subscriber: llvm-commits.
tstellarAMD accepted this revision.Jul 30 2015, 8:01 AM
tstellarAMD edited edge metadata.

This patch LGTM.

This revision is now accepted and ready to land.Jul 30 2015, 8:01 AM
arsenm closed this revision.Jul 30 2015, 10:04 AM

r243661