This is an archive of the discontinued LLVM Phabricator instance.

Clean up statepoint argument accessors.
ClosedPublic

Authored by pgavlin on May 8 2015, 1:45 PM.

Details

Summary

This change cleans up some assertions and removes the '# transition args' and '# deopt args' parameters from their respective iterators. These arguments are now accessed solely through 'getNumTotal{GCTransition,VMS}Args'.

Diff Detail

Event Timeline

pgavlin updated this revision to Diff 25368.May 8 2015, 1:45 PM
pgavlin retitled this revision from to Clean up statepoint argument accessors..
pgavlin updated this object.
pgavlin edited the test plan for this revision. (Show Details)
pgavlin added reviewers: reames, sanjoy.
pgavlin added a subscriber: Unknown Object (MLST).

Sanjoy, this might conflict with D9546. I'm happy to wait until that patch has landed before checking this in.

reames accepted this revision.May 12 2015, 10:34 AM
reames edited edge metadata.

LGTM. Thanks for the cleanup work!

This revision is now accepted and ready to land.May 12 2015, 10:34 AM
This revision was automatically updated to reflect the committed changes.