• marja's avatar
    Revert of Put Scopes into temporary Zone (patchset #12 id:220001 of... · 51685002
    marja authored
    Revert of Put Scopes into temporary Zone (patchset #12 id:220001 of https://codereview.chromium.org/2193793002/ )
    
    Reason for revert:
    Broke Node.js tests (test-require-dot etc.)
    
    Original issue's description:
    > Put Scopes into temporary Zone
    >
    > When parsing a eagerly-parsed-but-lazily-compiled function, we
    > used to put some of its AST nodes into a discardable Zone. This
    > CL puts the function Scope, its inner Scopes and the related AST
    > nodes (Declarations, VariableProxys) into the temporary Zone
    > too. This reduces peak memory usage and enables future work to
    > keep the temporary Zone around for later compilation.
    >
    > BUG=
    >
    > Committed: https://crrev.com/eaebdd858b466057ccc39894a172c9b66868e8f7
    > Cr-Commit-Position: refs/heads/master@{#38232}
    
    TBR=adamk@chromium.org,titzer@chromium.org
    # Skipping CQ checks because original CL landed less than 1 days ago.
    NOPRESUBMIT=true
    NOTREECHECKS=true
    NOTRY=true
    BUG=
    
    Review-Url: https://codereview.chromium.org/2205013002
    Cr-Commit-Position: refs/heads/master@{#38258}
    51685002
scopes.cc 53.1 KB