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. On 2010/06/16 15:20:27, Erik Corry wrote:
'even though the default is not to do it' doesn't make sense to me
here. Well I was trying to explain ASSERT(!ShouldGenerateSmiCode()); but failed. Rephrased the comment. http://codereview.chromium.org/2873001/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
