From aea684ede47369988a95cb52169753fc305e9f11 Mon Sep 17 00:00:00 2001 From: "ager@chromium.org" Date: Wed, 28 Oct 2009 10:19:56 +0000 Subject: [PATCH] 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 --- src/runtime.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/runtime.js b/src/runtime.js index 789bfdb..ba19871 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); } } -- 2.7.4