• Leszek Swirski's avatar
    [ast] Allocate cons strings on-demand · ddca3606
    Leszek Swirski authored
    Remove AstConsString "internalization", and instead make the conversion
    to heap String be on-demand with an Allocate method. We never actually
    need the heapified cons string more than once, so there's no need to do
    the internalization walk or do the next/string union dance in the
    AstConsString class.
    
    This also allows us to specify how we want to allocate the String at the
    call site. In particular, it allows us to allocate a flat SeqString rather
    rather than a ConsString. This allows us to avoid allocating ConsStrings
    which will just be passed to a flatten call, and especially avoid
    allocating dead ConsStrings in the off-thread old space.
    
    Bug: chromium:1011762
    Bug: chromium:1043168
    Change-Id: Id851f2f7529d92ad7e5388eb22823fd6d1959cd0
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2020953Reviewed-by: 's avatarToon Verwaest <verwaest@chromium.org>
    Reviewed-by: 's avatarUlan Degenbaev <ulan@chromium.org>
    Commit-Queue: Leszek Swirski <leszeks@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#66042}
    ddca3606
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...
test262 Loading commit data...
torque Loading commit data...
unittests Loading commit data...
wasm-api-tests Loading commit data...
wasm-js Loading commit data...
wasm-spec-tests Loading commit data...
webkit Loading commit data...
BUILD.gn Loading commit data...
OWNERS Loading commit data...