Reviewers: adamk,
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.
Recommitting this patch after a bug fix in Crankshaft to allow a
desugaring with certain elements missing a source location:
https://codereview.chromium.org/1313443002
BUG=v8:4377
LOG=Y
R=adamk
Please review this at https://codereview.chromium.org/1309163003/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+161, -15 lines):
M src/ast-value-factory.h
M src/parser.h
M src/parser.cc
M src/scopes.h
A test/mjsunit/regress/regress-4377.js
M test/mjsunit/switch.js
--
--
v8-dev mailing list
v8-dev@googlegroups.com
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 v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
