• adamk's avatar
    Don't treat catch scopes as possibly-shadowing for sloppy eval · 75f2d65f
    adamk authored
    Scope analysis is over-conservative when treating variable resolutions
    as possibly-shadowed by a sloppy eval. In the attached bug, this comes
    into play since catch scopes have different behavior with respect to
    the "calls eval" in eager vs lazy compilation (in the latter, they
    are never marked as "calls eval" because CatchContexts don't have
    an associated ScopeInfo).
    
    This patch changes the scope-type check to also eliminate a few other
    cases where shadowing isn't possible, such as non-declaration block scopes.
    
    BUG=chromium:608279
    LOG=n
    
    Review-Url: https://codereview.chromium.org/1950803002
    Cr-Commit-Position: refs/heads/master@{#36046}
    75f2d65f
Name
Last commit
Last update
..
OWNERS Loading commit data...
ast-expression-rewriter.cc Loading commit data...
ast-expression-rewriter.h Loading commit data...
ast-expression-visitor.cc Loading commit data...
ast-expression-visitor.h Loading commit data...
ast-literal-reindexer.cc Loading commit data...
ast-literal-reindexer.h Loading commit data...
ast-numbering.cc Loading commit data...
ast-numbering.h Loading commit data...
ast-value-factory.cc Loading commit data...
ast-value-factory.h Loading commit data...
ast.cc Loading commit data...
ast.h Loading commit data...
modules.cc Loading commit data...
modules.h Loading commit data...
prettyprinter.cc Loading commit data...
prettyprinter.h Loading commit data...
scopeinfo.cc Loading commit data...
scopeinfo.h Loading commit data...
scopes.cc Loading commit data...
scopes.h Loading commit data...
variables.cc Loading commit data...
variables.h Loading commit data...