LGTM

http://codereview.chromium.org/573056/diff/13002/11006
File src/codegen.cc (right):

http://codereview.chromium.org/573056/diff/13002/11006#newcode400
src/codegen.cc:400: //if (entry->nargs != new_entry->nargs) return
false;
Remove please.

http://codereview.chromium.org/573056/diff/13002/11007
File test/cctest/test-log-stack-tracer.cc (right):

http://codereview.chromium.org/573056/diff/13002/11007#newcode236
test/cctest/test-log-stack-tracer.cc:236: // _FastCharCodeAt is not used
in our tests.
Update comment?

http://codereview.chromium.org/573056

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

Reply via email to