LGTM.

http://codereview.chromium.org/596036/diff/4002/9005
File src/x64/ic-x64.cc (right):

http://codereview.chromium.org/596036/diff/4002/9005#newcode986
src/x64/ic-x64.cc:986: // rcx                      : function name
Make all the CallIC "S t a t e" comments the same (as long as the state
is the same).

http://codereview.chromium.org/596036/diff/4002/9002
File src/x64/virtual-frame-x64.cc (right):

http://codereview.chromium.org/596036/diff/4002/9002#newcode1022
src/x64/virtual-frame-x64.cc:1022: // Function name, arguments, and
receiver, and function name are found
Function name listed twice.

http://codereview.chromium.org/596036/diff/4002/9003
File src/x64/virtual-frame-x64.h (right):

http://codereview.chromium.org/596036/diff/4002/9003#newcode350
src/x64/virtual-frame-x64.h:350: // Call call IC.  Function name,
arguments, and receiver, and function
This comment mentions function name twice.

http://codereview.chromium.org/596036

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

Reply via email to