From: ager@chromium.org Date: Wed, 28 Oct 2009 10:19:56 +0000 (+0000) Subject: Add fast check for NaN in NumberCompare slow case. X-Git-Tag: upstream/4.7.83~23060 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=aea684ede47369988a95cb52169753fc305e9f11;p=platform%2Fupstream%2Fv8.git Add fast check for NaN in NumberCompare slow case. Review URL: http://codereview.chromium.org/344010 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@3158 ce2b1a6d-e550-0410-aec6-3dcde31c8c00 --- diff --git a/src/runtime.js b/src/runtime.js index 789bfdb77..ba19871f1 100644 --- a/src/runtime.js +++ b/src/runtime.js @@ -128,7 +128,10 @@ function COMPARE(x, ncr) { if (IS_STRING(a) && IS_STRING(b)) { return %StringCompare(a, b); } else { - return %NumberCompare(%ToNumber(a), %ToNumber(b), ncr); + var a_number = %ToNumber(a); + var b_number = %ToNumber(b); + if (NUMBER_IS_NAN(a_number) || NUMBER_IS_NAN(b_number)) return ncr; + return %NumberCompare(a_number, b_number, ncr); } }