Comment #1 on issue 4377 by [email protected]: Switch should introduce
a scope
https://code.google.com/p/v8/issues/detail?id=4377#c1
The following revision refers to this bug:
https://chromium.googlesource.com/v8/v8.git/+/9edbc1f21eb1050cabbe3b8bc9aebf89ada7ebd7
commit 9edbc1f21eb1050cabbe3b8bc9aebf89ada7ebd7
Author: littledan <[email protected]>
Date: Fri Aug 21 23:54:21 2015
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
Review URL: https://codereview.chromium.org/1293283002
Cr-Commit-Position: refs/heads/master@{#30314}
[modify]
http://crrev.com/9edbc1f21eb1050cabbe3b8bc9aebf89ada7ebd7/src/ast-value-factory.h
[modify]
http://crrev.com/9edbc1f21eb1050cabbe3b8bc9aebf89ada7ebd7/src/parser.cc
[modify]
http://crrev.com/9edbc1f21eb1050cabbe3b8bc9aebf89ada7ebd7/src/parser.h
[modify]
http://crrev.com/9edbc1f21eb1050cabbe3b8bc9aebf89ada7ebd7/src/scopes.h
[add]
http://crrev.com/9edbc1f21eb1050cabbe3b8bc9aebf89ada7ebd7/test/mjsunit/regress/regress-4377.js
[modify]
http://crrev.com/9edbc1f21eb1050cabbe3b8bc9aebf89ada7ebd7/test/mjsunit/switch.js
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
--
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.
