Comment #3 on issue 4377 by [email protected]: Switch should introduce
a scope
https://code.google.com/p/v8/issues/detail?id=4377#c3
The following revision refers to this bug:
https://chromium.googlesource.com/v8/v8.git/+/31b801802997ed54117c7f2bc55f86e9e032cf3e
commit 31b801802997ed54117c7f2bc55f86e9e032cf3e
Author: littledan <[email protected]>
Date: Sat Aug 22 01:24:18 2015
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
Review URL: https://codereview.chromium.org/1309043004
Cr-Commit-Position: refs/heads/master@{#30316}
[modify]
http://crrev.com/31b801802997ed54117c7f2bc55f86e9e032cf3e/src/ast-value-factory.h
[modify]
http://crrev.com/31b801802997ed54117c7f2bc55f86e9e032cf3e/src/parser.cc
[modify]
http://crrev.com/31b801802997ed54117c7f2bc55f86e9e032cf3e/src/parser.h
[modify]
http://crrev.com/31b801802997ed54117c7f2bc55f86e9e032cf3e/src/scopes.h
[delete]
http://crrev.com/9c79e69e6c9725501852a26de1d0284e6aad038d/test/mjsunit/regress/regress-4377.js
[modify]
http://crrev.com/31b801802997ed54117c7f2bc55f86e9e032cf3e/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.
