• Jakob Gruber's avatar
    [regexp] Release regexp zone memory during JS parsing · 30cab7b1
    Jakob Gruber authored
    Since early regexp errors were implemented in
    crrev.com/a56874d3, the JS parser
    calls into the regexp parser to validate the regexp literal syntax.
    
    For these calls, the JS parser passes its Zone to the regexp parser.
    This means that scripts with multiple regexp literals are all parsed
    using the same Zone memory. Very large scripts with many (think
    hundreds of thousands) regexp literals may thus run out of memory
    whereas previously they would parse and run successfully.
    
    This CL fixes the OOMs by resetting the state of the JS parser Zone
    around regexp parser calls. We introduce a new ZoneScope class,
    similar to HandleScope, which controls the lifetime of zone objects
    allocated within its scope. In other words:
    
     {
       ZoneScope zone_scope(zone);  // Store zone state S.
       // ... Allocate objects O in zone.
       // zone is now in state S'.
     }
     // zone_scope goes out of scope, reset zone to state S. Objects O
     // are freed and no longer usable.
    
    Fixed: chromium:1264014
    Bug: v8:896
    Change-Id: I3e7ac36f25a9d6c4eda2460bd1bea9814685e89b
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3256783Reviewed-by: 's avatarToon Verwaest <verwaest@chromium.org>
    Commit-Queue: Jakob Gruber <jgruber@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#77646}
    30cab7b1
Name
Last commit
Last update
.github Loading commit data...
bazel Loading commit data...
build_overrides Loading commit data...
custom_deps Loading commit data...
docs Loading commit data...
gni Loading commit data...
include Loading commit data...
infra Loading commit data...
samples Loading commit data...
src Loading commit data...
test Loading commit data...
testing Loading commit data...
third_party Loading commit data...
tools Loading commit data...
.bazelrc Loading commit data...
.clang-format Loading commit data...
.clang-tidy Loading commit data...
.editorconfig Loading commit data...
.flake8 Loading commit data...
.git-blame-ignore-revs Loading commit data...
.gitattributes Loading commit data...
.gitignore Loading commit data...
.gn Loading commit data...
.mailmap Loading commit data...
.vpython Loading commit data...
.ycm_extra_conf.py Loading commit data...
AUTHORS Loading commit data...
BUILD.bazel Loading commit data...
BUILD.gn Loading commit data...
CODE_OF_CONDUCT.md Loading commit data...
COMMON_OWNERS Loading commit data...
DEPS Loading commit data...
DIR_METADATA Loading commit data...
ENG_REVIEW_OWNERS Loading commit data...
INFRA_OWNERS Loading commit data...
INTL_OWNERS Loading commit data...
LICENSE Loading commit data...
LICENSE.fdlibm Loading commit data...
LICENSE.strongtalk Loading commit data...
LICENSE.v8 Loading commit data...
LOONG_OWNERS Loading commit data...
MIPS_OWNERS Loading commit data...
OWNERS Loading commit data...
PPC_OWNERS Loading commit data...
PRESUBMIT.py Loading commit data...
README.md Loading commit data...
RISCV_OWNERS Loading commit data...
S390_OWNERS Loading commit data...
WATCHLISTS Loading commit data...
WORKSPACE Loading commit data...
codereview.settings Loading commit data...