• adamk's avatar
    Include harmony-string-padding.js in the context snapshot · 02149d89
    adamk authored
    The attached chromium bug suggests that turning this on via a flag
    regressed a perf test. Having previously run into slowdowns due
    to experimental JS natives, my suspicion is that this regression
    can be fixed by including the new builtins in the snapshot.
    My intention is to land this patch to see if the regression
    goes away on the bots.
    
    Given the nature of this feature, it seems low-risk to remove
    the runtime flag. If need be, it would be trivial to craft
    a patch to remove these two methods from String.prototype
    if we run into web compat issues during the release (note
    that no release branch has yet picked up this feature).
    
    BUG=v8:4954, chromium:677444
    
    Review-Url: https://codereview.chromium.org/2610853010
    Cr-Commit-Position: refs/heads/master@{#42111}
    02149d89
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...
mozilla Loading commit data...
preparser Loading commit data...
promises-aplus Loading commit data...
simdjs Loading commit data...
test262 Loading commit data...
unittests Loading commit data...
webkit Loading commit data...
BUILD.gn Loading commit data...
bot_default.gyp Loading commit data...
bot_default.isolate Loading commit data...
default.gyp Loading commit data...
default.isolate Loading commit data...
optimize_for_size.gyp Loading commit data...
optimize_for_size.isolate Loading commit data...
perf.gyp Loading commit data...
perf.isolate Loading commit data...