Issue metadata
Sign in to add a comment
|
0.5%-1.5% regression in sizes at 558397:558406 |
||||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
May 18 2018
For this one, the bisect said it could not select a test. The Bisect button is grayed so can't trigger manually either. Are these tests no longer? +grt@ to confirm, +sullivan@ in case we need to remove alerts of this kind.
,
May 18 2018
A couple of things happening here: * The sizes tests are run on the builders, not the testers, so they are not bisectable. * We build every chromium revision on the builders, so generally this isn't an issue. * If there's just a single revision in the CL range, the dashboard is supposed to auto-assign the bug to the revision author. * In this case, the dip after the original bump seems to have confused the dashboard into putting the alerts in the wrong place. I think this also confused it into thinking there's not a single revision responsible? I'll fix it, but the summary is that this should be assigned to thakis because all the alerts came from r558402, "Reland "Reland "win: Link with lld instead of link.exe by default""" thakis, is the binary size increase expected?
,
May 19 2018
It's not expected, but we learned about it when we landed the switch an earlier time. The switch is currently reverted, and we've since fixed the issue upstream and it won't reappear the next time we reland the switch :-) |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, May 18 2018