LGTM

Are there no tests in the es5conform test suite that we start passing because of
this change?


http://codereview.chromium.org/6523052/diff/10002/src/ic.cc
File src/ic.cc (right):

http://codereview.chromium.org/6523052/diff/10002/src/ic.cc#newcode442
src/ic.cc:442: if (function->shared()->strict_mode() ||
Does this fit on one line?

http://codereview.chromium.org/6523052/diff/10002/src/ic.cc#newcode519
src/ic.cc:519: // functions do not wrap the receiver, non-strict
functions do.
non-strict functions and objects called as functions do?

http://codereview.chromium.org/6523052/diff/10002/src/ic.cc#newcode813
src/ic.cc:813: // functions do not wrap the receiver, non-strict
functions do.
Ditto.

http://codereview.chromium.org/6523052/

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to