• peterwmwong's avatar
    [builtins] Add Array.p.join fastpath for single element array. · 986aa36b
    peterwmwong authored
    This is resurrecting an optimization from the pre-torque version.
    Avoid allocating another sequential string for the result when there's only one
    element, just return the ToString-ed element.  This not only saves time writing
    to this destination string, but also reduce GC pressure.
    
    The System Health Memory Benchmark (load:media:google_images) exposed this missing
    optimization with a 15% regression in memory usage.  Very large external strings
    were being copied into V8's heap as sequential string.
    
    Bug: chromium:896612
    Change-Id: Ieb61906f64100cdc15bf96f3ebcccb1207f75356
    Reviewed-on: https://chromium-review.googlesource.com/c/1316620Reviewed-by: 's avatarJakob Gruber <jgruber@chromium.org>
    Commit-Queue: Peter Wong <peter.wm.wong@gmail.com>
    Cr-Commit-Position: refs/heads/master@{#57241}
    986aa36b
Name
Last commit
Last update
..
benchmarks Loading commit data...
cctest Loading commit data...
common Loading commit data...
debugger Loading commit data...
fuzzer Loading commit data...
inspector Loading commit data...
intl Loading commit data...
js-perf-test Loading commit data...
memory Loading commit data...
message Loading commit data...
mjsunit Loading commit data...
mkgrokdump Loading commit data...
mozilla Loading commit data...
preparser Loading commit data...
test262 Loading commit data...
torque Loading commit data...
unittests Loading commit data...
wasm-js Loading commit data...
wasm-spec-tests Loading commit data...
webkit Loading commit data...
BUILD.gn Loading commit data...