LGTM
http://codereview.chromium.org/2873001/diff/1/2 File src/arm/codegen-arm.cc (right): http://codereview.chromium.org/2873001/diff/1/2#newcode8172 src/arm/codegen-arm.cc:8172: // support for modulus checking for smis makes sense. 'even though the default is not to do it' doesn't make sense to me here. http://codereview.chromium.org/2873001/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
