Reviewers: rossberg, arv,
Message:
This is a reapplication of https://codereview.chromium.org/1097283003. The
first patch is just the rebased original patch. The second version adds the
nosnap test fix, along with mips/mips64 backend support.
Description:
Resolve references to "this" the same way as normal variables
Make the parser handle references to "this" as unresolved variables, so the
same logic as for the rest of function parameters is used for the receiver.
Minor additions to the code generation handle copying the receiver to the
context, along with the rest of the function parameters.
Based on work by Adrian Perez de Castro <[email protected]>.
BUG=v8:2700
LOG=N
Please review this at https://codereview.chromium.org/1130733003/
Base URL: https://chromium.googlesource.com/v8/v8@master
Affected files (+313, -120 lines):
M src/arm/full-codegen-arm.cc
M src/arm/lithium-codegen-arm.cc
M src/arm64/full-codegen-arm64.cc
M src/arm64/lithium-codegen-arm64.cc
M src/compiler/ast-graph-builder.h
M src/compiler/ast-graph-builder.cc
M src/contexts.cc
M src/heap/heap.h
M src/ia32/full-codegen-ia32.cc
M src/ia32/lithium-codegen-ia32.cc
M src/mips/full-codegen-mips.cc
M src/mips/lithium-codegen-mips.cc
M src/mips64/full-codegen-mips64.cc
M src/mips64/lithium-codegen-mips64.cc
M src/parser.cc
M src/runtime/runtime-debug.cc
M src/runtime/runtime-scopes.cc
M src/scopeinfo.cc
M src/scopes.h
M src/scopes.cc
M src/variables.cc
M src/x64/full-codegen-x64.cc
M src/x64/lithium-codegen-x64.cc
M test/cctest/test-serialize.cc
M test/mjsunit/debug-scopes.js
A test/mjsunit/harmony/arrow-functions-this.js
--
--
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.
