LGTM

http://codereview.chromium.org/7477045/diff/59001/src/x64/code-stubs-x64.cc
File src/x64/code-stubs-x64.cc (right):

http://codereview.chromium.org/7477045/diff/59001/src/x64/code-stubs-x64.cc#newcode2377
src/x64/code-stubs-x64.cc:2377: __ Set(r14, 0);
On 2011/08/26 10:44:40, Yang wrote:
On 2011/08/25 16:02:45, antonm wrote:
> maybe use arg1?

I don't quite follow. arg1 is not defined until way later.

Never mind---my fault.  Sorry.

http://codereview.chromium.org/7477045/

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

Reply via email to