http://codereview.chromium.org/7830036/diff/1/src/v8natives.js File src/v8natives.js (right):
http://codereview.chromium.org/7830036/diff/1/src/v8natives.js#newcode96 src/v8natives.js:96: return !NUMBER_IS_NAN(number) && number !== (1 / 0) && number !== (-1 / 0); !NUMBER_IS_NAN(number) is equivalent to %_IsSmi(%IS_VAR(number)) || (number == number), but the double-untag before the cmp-id-and-branch (the compiled equality check) already does the smi check. Therefore I propose to replace (every?) !NUMBER_IS_NAN(foo) with (foo === foo). This gains another 5% speed for issue 1657. http://codereview.chromium.org/7830036/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
