Still LGTM
http://codereview.chromium.org/4456002/diff/83002/47007 File src/ia32/stub-cache-ia32.cc (right): http://codereview.chromium.org/4456002/diff/83002/47007#newcode416 src/ia32/stub-cache-ia32.cc:416: // Amount of pointers to be reserved on stack for v8::Arguments::implicit_args_. I am not a native speaker, but I would rather say 'number of pointers' http://codereview.chromium.org/4456002/diff/83002/47007#newcode416 src/ia32/stub-cache-ia32.cc:416: // Amount of pointers to be reserved on stack for v8::Arguments::implicit_args_. 'v8::Arguments::implicit_args_': should either be 'for fast API call' or constant renamed back, imho http://codereview.chromium.org/4456002/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
