Issue metadata
Sign in to add a comment
|
2.4% regression in system_health.memory_desktop at 591073:591215 |
||||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Sep 19
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/170c2484e40000
,
Sep 20
📍 Found a significant difference after 1 commit. https://pinpoint-dot-chromeperf.appspot.com/job/170c2484e40000 [objects] Change String::length field to uint32_t. by bmeurer@chromium.org https://chromium.googlesource.com/v8/v8/+/c7a0049e1b70e8f455229c8a470df605ea064f70 5.183e+08 → 5.279e+08 (+9.663e+06) Understanding performance regressions: http://g.co/ChromePerformanceRegressions Benchmark documentation link: https://bit.ly/system-health-benchmarks
,
Sep 20
The CL in question reduces the size of every String and Symbol by one word (on 64-bit architectures). 32-bit architectures are not affected memory-wise. Otherwise there are no memory-related changes in the CL. So this seems to be another of these cases where GC pressure goes down and thereby V8 effectively consumes more memory. Not sure if the GC team wants to take a look, but marking as WontFix from my side. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Sep 19