Open Bug 1464742 Opened 3 years ago Updated 3 years ago
Code size regressed by porting Inspector
Utils::Get CSSValues For Property to use Servo
See discussions in bug 1434130 comment 102 and after.
(In reply to Xidorn Quan [:xidorn] UTC+10 from comment #108) > Filed bug 1464742 for that. From the graph it doesn't seem to be a very > significant regression, and also I don't have a great idea on how it can be > fixed, so I gave it a P3 for now. We can continue discussion there. Unfortunately after we merged ICU's data into libxul (bug 1445524) it really skewed our numbers making sizeable regressions look "smaller". Focusing on the raw numbers makes more sense -- a 212k regression is sizeable. Zooming in  makes it a little more clear. From a Fission point of view I'm mainly concerned about increases in non-shareable data, so .data, .data.rel.ro, .bss etc. We're tracking those now  but don't have historical data back this far. In this case there's a 33,872 byte increase in .data.rel.ro. That's not huge, but it would be nice to understand what's going on and get it lower if possible.  https://treeherder.mozilla.org/perf.html#/graphs?timerange=7776000&series=autoland,1338582,1,2&series=mozilla-inbound,1299711,1,2&zoom=1524928657766.8357,1525000662347.2383,129956760.51003662,130707928.29188234&selected=autoland,1338582,331976,460069505,2  https://treeherder.mozilla.org/perf.html#/graphs?timerange=7776000&series=mozilla-inbound,1686770,1,2
(In reply to Eric Rahm [:erahm] (please no mozreview requests) from comment #1) > In this case there's a 33,872 byte increase in .data.rel.ro. That's not > huge, but it would be nice to understand what's going on and get it lower if > possible. I think every data added in that change should fully be shareable, so it would be very useful to know what ends up being in .data.rel.ro there. Maybe trait function pointers?
You need to log in before you can comment on or make changes to this bug.