LGTM with a couple of nits.
http://codereview.chromium.org/515012/diff/1/4 File src/arm/virtual-frame-arm.cc (right): http://codereview.chromium.org/515012/diff/1/4#newcode149 src/arm/virtual-frame-arm.cc:149: if (count < 5) { Please introduce a platform-dependent named constant for this value. http://codereview.chromium.org/515012/diff/1/4#newcode155 src/arm/virtual-frame-arm.cc:155: // For more kocals a loop in generated code is more compact. kocals -> locals http://codereview.chromium.org/515012/diff/1/2 File src/ia32/virtual-frame-ia32.cc (right): http://codereview.chromium.org/515012/diff/1/2#newcode543 src/ia32/virtual-frame-ia32.cc:543: for (int i = 0; i < count; i++) { This loop should be extracted. It happens in all branches, so you can just always do it at the end. http://codereview.chromium.org/515012/diff/1/3 File src/x64/virtual-frame-x64.cc (right): http://codereview.chromium.org/515012/diff/1/3#newcode155 src/x64/virtual-frame-x64.cc:155: for (int i = 0; i < count; i++) { Ditto. http://codereview.chromium.org/515012 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
