Bug 1917031 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Over [here](https://bugzilla.mozilla.org/show_bug.cgi?id=1892764#c10) I experimented with making all wrappers "ProbablyShortLived" on top of the DOM Allocation Site patch. 

This showed some nice improvements (up to 10% on the ReactSVG subtest). Which makes me think maybe once we have the DOM allocationSites we can consider removing this annotation and marking everything as nursery allocatable, relying on the allocSite to pre-tenure where possible.

Now, there are some regressions too that probably need to be considered more deeply. It may be that we actually need to flip the sense of the annotation from "ProbablyShortLived" to "ProbablyLongLived"
Over [here](https://bugzilla.mozilla.org/show_bug.cgi?id=1892764#c10) I experimented with making all wrappers "ProbablyShortLivingWrapper" on top of the DOM Allocation Site patch. 

This showed some nice improvements (up to 10% on the ReactSVG subtest). Which makes me think maybe once we have the DOM allocationSites we can consider removing this annotation and marking everything as nursery allocatable, relying on the allocSite to pre-tenure where possible.

Now, there are some regressions too that probably need to be considered more deeply. It may be that we actually need to flip the sense of the annotation from "ProbablyShortLivingWrapper" to "ProbablyLongLivingWrapper"

Back to Bug 1917031 Comment 0