Reviewers: Jakob,
https://codereview.chromium.org/12263004/diff/1/test/mjsunit/compiler/inline-function-apply.js
File test/mjsunit/compiler/inline-function-apply.js (right):
https://codereview.chromium.org/12263004/diff/1/test/mjsunit/compiler/inline-function-apply.js#newcode51
test/mjsunit/compiler/inline-function-apply.js:51: // TODO(2539): Enable
the mutation below once bug is fixed.
Note that this is a bug that is already present in our current
optimization and is not introduced by this change. This test just
flushed it out.
Description:
Allow full inlining of f.apply(this, arguments) calls.
This allows Crankshaft to completely inline a f.apply() dispatch if the
exact number of arguments is known and the function is constant. The
deoptimizer doesn't generate the f.apply() frame during deoptimization,
so the materialized frames look like f.apply() did a tailcall.
[email protected]
TEST=mjsunit/compiler/inline-function-apply
Please review this at https://codereview.chromium.org/12263004/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files:
M src/hydrogen.h
M src/hydrogen.cc
A + test/mjsunit/compiler/inline-function-apply.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/groups/opt_out.
