http://codereview.chromium.org/549147/diff/1/2 File src/arm/codegen-arm.cc (right):
http://codereview.chromium.org/549147/diff/1/2#newcode5276 src/arm/codegen-arm.cc:5276: if (CpuFeatures::IsSupported(VFP3) && Token::MOD != operation) { On 2010/01/26 09:05:26, Erik Corry wrote:
Since this 'if' is used several times I think it would express the
intent
clearer to have a bool variable with an explanatory name. eg. float_values_in_vfp_regs.
Done. http://codereview.chromium.org/549147/diff/1/2#newcode5426 src/arm/codegen-arm.cc:5426: // Convert smi in r1 to double in d6 On 2010/01/26 09:05:26, Erik Corry wrote:
Missing full stop.
Done. http://codereview.chromium.org/549147 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
