Reviewers: adamk,
Message:
Created Revert of Add a separate scope for switch
Description:
Revert of Add a separate scope for switch (patchset #7 id:120001 of
https://codereview.chromium.org/1293283002/ )
Reason for revert:
Breaks cctest/test-cpu-profiler/SourceLocation on nosnap
Original issue's description:
Add a separate scope for switch
The ES2015 specification for switch statements 13.12.11 specifies that
they get their own lexical scope. This patch introduces such a scope
through a complex desugaring in terms of blocks, done so that Crankshaft
does not have to be updated to support multiple constructs providing
scopes.
BUG=v8:4377
LOG=Y
R=adamk
Committed: https://crrev.com/9edbc1f21eb1050cabbe3b8bc9aebf89ada7ebd7
Cr-Commit-Position: refs/heads/master@{#30314}
[email protected]
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:4377
Please review this at https://codereview.chromium.org/1309043004/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+16, -162 lines):
M src/ast-value-factory.h
M src/parser.h
M src/parser.cc
M src/scopes.h
D test/mjsunit/regress/regress-4377.js
M test/mjsunit/switch.js
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
