Page MenuHomePhabricator

[GC] Remove redundant entiries in stackmap section (and test it this time)
ClosedPublic

Authored by reames on Mar 11 2020, 12:25 PM.

Details

Summary

This is a reimplementation of the optimization removed in D75964. The actual spill/fill optimization is handled by D76013, this one just worries about reducing the stackmap section size itself by eliminating redundant entries. As noted in the comments, we could go a lot further here, but avoiding the degenerate invoke case as we did before is probably "enough" in practice.

Diff Detail

Event Timeline

reames created this revision.Mar 11 2020, 12:25 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 11 2020, 12:25 PM
reames updated this revision to Diff 249725.Mar 11 2020, 12:26 PM

Correct diff this time..

skatkov accepted this revision.Mar 11 2020, 8:24 PM

LGTM

This revision is now accepted and ready to land.Mar 11 2020, 8:24 PM
This revision was automatically updated to reflect the committed changes.