Reviewers: Lasse Reichstein, Mads Ager,
http://codereview.chromium.org/6625048/diff/1/src/ast.h
File src/ast.h (right):
http://codereview.chromium.org/6625048/diff/1/src/ast.h#newcode1689
src/ast.h:1689: contains_loops_(contains_loops),
Strict mode is now needed in scopes.cc so I had to move it from
temporary scope + function literal to scope.
It is actually cleaner that way but with the downside of having multiple
scopes per function in the future and strict mode flag is per function
...
Description:
Strict mode arguments do not share binding with formal parameters.
Move strict mode flag from TemporaryScope to Scope so that it can be
accessed
from variable binding code.
Arguments do not alias in strict mode (ia32, x64 and arm, codegen and full
codegen).
Hydrogen tolerates null arguments_shadow().
In codegen-<arch> arguments object is allocated eagerly to capture values
before
they get modified.
BUG=
TEST=test/mjsunit/strict-mode.js
Please review this at http://codereview.chromium.org/6625048/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files:
M src/arm/codegen-arm.cc
M src/arm/full-codegen-arm.cc
M src/ast-inl.h
M src/ast.h
M src/hydrogen.cc
M src/ia32/codegen-ia32.cc
M src/ia32/full-codegen-ia32.cc
M src/parser.cc
M src/scopes.h
M src/scopes.cc
M src/x64/codegen-x64.cc
M src/x64/full-codegen-x64.cc
M test/es5conform/es5conform.status
M test/mjsunit/strict-mode.js
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
