• jarin's avatar
    Make non-Module generators only context allocate parameters. · a957b0f4
    jarin authored
    In particular, local variables should be allocated on stack (in bytecode register), and stored/loaded to the generator object on generator suspend/resume.
    
    The CL is based on @adamk's change to scoping/parsers (https://chromium-review.googlesource.com/c/498538/), I only made the debugger cope with this change.
    
    I should note that the CL changes the scope type of suspended generators from ScopeType.Closure to ScopeType.Local. In the future we might want to introduce ScopeType.SuspendedGenerator to make the distinction explicit.
    
    Some of the changes in the tests have been made because the debugger functions do not return scopes of closed generators anymore. Generators should be allowed to throw away their internal state when they finish.
    
    BUG=v8:6368
    
    Review-Url: https://codereview.chromium.org/2898163002
    Cr-Commit-Position: refs/heads/master@{#45515}
    a957b0f4
async-function-debug-evaluate.js 4.29 KB