Reviewers: rossberg, wingo, Michael Starzinger, Dmitry Lomov (chromium),
marja,
Message:
On 2014/07/14 13:55:47, rossberg wrote:
LGTM, only nits
The new patch set should have all the nits fixed. As I am not a committer,
could
you please land this for me?
Description:
Implement basic code generation for arrow functions
Implements code generation for arrow functions by desugaring them into
a FunctionLiteral. For the moment, a normal FUNCTION_SCOPE is used, so
"this" and "arguments" behave as in normal functions. Implementing the
correct scoping rules is to be done later on.
BUG=v8:2700
LOG=
Please review this at https://codereview.chromium.org/382893003/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+238, -145 lines):
M src/ast.h
M src/compiler.cc
M src/factory.h
M src/factory.cc
M src/full-codegen.cc
M src/objects.h
M src/objects-inl.h
M src/parser.h
M src/parser.cc
M src/preparser.h
M src/preparser.cc
M src/runtime.cc
M src/v8natives.js
M test/cctest/test-parsing.cc
A test/mjsunit/harmony/arrow-functions.js
M tools/generate-runtime-tests.py
--
--
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.
