LGTM
http://codereview.chromium.org/3143032/diff/14001/2002 File src/builtins.cc (right): http://codereview.chromium.org/3143032/diff/14001/2002#newcode328 src/builtins.cc:328: for (int i = 0; i < to_trim; i++) { you may save one store here as elms->address() would be overwritten with filler map in next store. And actually more as you could skip the whole header, but it might be too subtle to reason about. http://codereview.chromium.org/3143032/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
