Issue metadata
Sign in to add a comment
|
2.1%-8.8% regression in memory.desktop at 594269:594296 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Oct 2
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/16dd6308e40000
,
Oct 3
📍 Found significant differences after each of 3 commits. https://pinpoint-dot-chromeperf.appspot.com/job/16dd6308e40000 [Intl] Ship Intl.RelativeTimeFormat by ftang@chromium.org https://chromium.googlesource.com/v8/v8/+/a3b97babfb19490a948a931b3966cf4ff8c2a2bd memory:chrome:all_processes:reported_by_chrome:v8:effective_size: 3.327e+06 → 3.595e+06 (+2.68e+05) [Intl] move DateTimeFormat.prototype.resolvedOptions to c++ by ftang@chromium.org https://chromium.googlesource.com/v8/v8/+/d34ffa6a3822fb7ca73ea0b6f7c6f8a27ea0fd79 memory:chrome:all_processes:reported_by_chrome:v8:effective_size: 3.567e+06 → 3.595e+06 (+2.79e+04) Revert "[interpreter] Separate bytecodes for one-shot property loads and stores" by mslekova@chromium.org https://chromium.googlesource.com/v8/v8/+/3c3330f6f0cae9a4c894cbf030d742c25358a1db memory:chrome:all_processes:reported_by_chrome:v8:effective_size: 3.595e+06 → 3.594e+06 (-1334) Understanding performance regressions: http://g.co/ChromePerformanceRegressions Benchmark documentation link: None
,
Oct 4
The memory regression was due to a CL reverted, which is now relanded as https://chromium.googlesource.com/v8/v8/+/1bff2e991e5a2d42dc6d24d85ffc70865ef5ab64 This should fix the regression once the CL is rolled into Chromium. |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Oct 2