Issue metadata
Sign in to add a comment
|
5.8%-5.9% regression in memory.top_10_mobile at 546583:546715 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Apr 2 2018
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/14dfe77f440000
,
Apr 3 2018
📍 Found significant differences after each of 2 commits. https://pinpoint-dot-chromeperf.appspot.com/job/14dfe77f440000 [Liftoff][mips] Implement instruction for type conversions by sreten.kovacevic@mips.com https://chromium.googlesource.com/v8/v8/+/53decc1a03f5659fe64c9956f3ba6b471c971bdd Remove legacy C++ implementation of Array#slice by danno@chromium.org https://chromium.googlesource.com/v8/v8/+/5020630808043615f2f6fbbfba1ea73342bebd69 Understanding performance regressions: http://g.co/ChromePerformanceRegressions
,
Dec 3
This CL simply removed dead code, there was not change in functionality, so it's unlikely that it was actually this CL. There were a bunch of other bigger functionality changes in this VL roll that seem more likely culprits. However, since the memory graphs have long-since recovered, I'm going to close as WontFix. |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Apr 2 2018