• 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
..
console Loading commit data...
cpu-profiler Loading commit data...
debugger Loading commit data...
heap-profiler Loading commit data...
runtime Loading commit data...
sessions Loading commit data...
type-profiler Loading commit data...
BUILD.gn Loading commit data...
DEPS Loading commit data...
OWNERS Loading commit data...
inspector-test.cc Loading commit data...
inspector.status Loading commit data...
isolate-data.cc Loading commit data...
isolate-data.h Loading commit data...
json-parse-expected.txt Loading commit data...
json-parse.js Loading commit data...
protocol-test.js Loading commit data...
task-runner.cc Loading commit data...
task-runner.h Loading commit data...
testcfg.py Loading commit data...