Comment #5 on issue 4377 by [email protected]: Switch should introduce
a scope
https://code.google.com/p/v8/issues/detail?id=4377#c5
The following revision refers to this bug:
https://chromium.googlesource.com/v8/v8.git/+/45e2628df7bc4151a893f38794014082e0b269d5
commit 45e2628df7bc4151a893f38794014082e0b269d5
Author: littledan <[email protected]>
Date: Mon Aug 24 18:57:08 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.
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
Review URL: https://codereview.chromium.org/1309163003
Cr-Commit-Position: refs/heads/master@{#30340}
[modify]
http://crrev.com/45e2628df7bc4151a893f38794014082e0b269d5/src/ast-value-factory.h
[modify]
http://crrev.com/45e2628df7bc4151a893f38794014082e0b269d5/src/parser.cc
[modify]
http://crrev.com/45e2628df7bc4151a893f38794014082e0b269d5/src/parser.h
[modify]
http://crrev.com/45e2628df7bc4151a893f38794014082e0b269d5/src/scopes.h
[add]
http://crrev.com/45e2628df7bc4151a893f38794014082e0b269d5/test/mjsunit/regress/regress-4377.js
[modify]
http://crrev.com/45e2628df7bc4151a893f38794014082e0b269d5/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.
