3%-22.6% regression in speedometer2 at 526209:526235 |
|||||||||
Issue descriptionSee the link to graphs below.
,
Jan 2 2018
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/14db1c4d040000
,
Jan 2 2018
😿 Pinpoint job stopped with an error. https://pinpoint-dot-chromeperf.appspot.com/job/14db1c4d040000
,
Jan 3 2018
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/14b53d03040000
,
Jan 3 2018
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/12e07503040000
,
Jan 3 2018
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/12cf386d040000
,
Jan 3 2018
😿 Pinpoint job stopped with an error. https://pinpoint-dot-chromeperf.appspot.com/job/14b53d03040000
,
Jan 3 2018
😿 Pinpoint job stopped with an error. https://pinpoint-dot-chromeperf.appspot.com/job/12cf386d040000
,
Jan 3 2018
I think based on https://pinpoint-dot-chromeperf.appspot.com/job/14db1c4d040000, https://crrev.com/d5fbf7c5c3f8f9b46b75f674771f3533c7e3e24d is the culprit. I was able to reproduce the regression locally as well.
,
Jan 4 2018
Franzi, can you please investigate?
,
Jan 4 2018
,
Jan 4 2018
Sorry about that, seems likely that the interceptor change causes performances regressions. I reverted the CL for now since it's a blocker and we'll investigate.
,
Jan 4 2018
Please add affected OSs.
,
Jan 5 2018
Issue 798286 has been merged into this issue.
,
Jan 5 2018
,
Jan 16 2018
franzih@ Ping! This issue is marked as RB-Beta, could you please let us know is there any latest update available on this issue?
,
Jan 16 2018
I reverted the problematic commit last week, it's in 65. https://chromium.googlesource.com/v8/v8/+/971875ca4d4ff82bc93fa906eb58a208fa9a3eeb This should have fixed the performance regression.
,
Jan 18 2018
franzih@ If this issue is fixed completely, could you please change the status of the bug. Thanks!
,
Jan 18 2018
,
Jan 18 2018
[Auto-generated comment by a script] We noticed that this issue is targeted for M-65; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-65 label, otherwise remove Merge-TBD label. Thanks.
,
Jan 19 2018
Per comment #17, this change is already in M65. No merge is needed.
,
Jul 17
|
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Jan 2 2018