LGTM

http://codereview.chromium.org/509007/diff/1/3
File src/ia32/codegen-ia32.cc (right):

http://codereview.chromium.org/509007/diff/1/3#newcode6130
src/ia32/codegen-ia32.cc:6130: __ mov(scratch.reg(),
FieldOperand(lhs.reg(), HeapNumber::kExponentOffset));
Line longer than 80 chars.

http://codereview.chromium.org/509007/diff/1/3#newcode6132
src/ia32/codegen-ia32.cc:6132: not_a_number.Branch(above_equal, &lhs);
// It's a negative NaN or -Infinity.
Ditto.

http://codereview.chromium.org/509007

-- 
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to