Reviewers: Mads Ager,

Description:
Expand fast case of parseInt to include radix == 10 and radix == 0.

Please review this at http://codereview.chromium.org/6532087/

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
  M     src/v8natives.js


Index: src/v8natives.js
===================================================================
--- src/v8natives.js    (revision 6877)
+++ src/v8natives.js    (working copy)
@@ -92,7 +92,7 @@

 // ECMA-262 - 15.1.2.2
 function GlobalParseInt(string, radix) {
-  if (IS_UNDEFINED(radix)) {
+  if (IS_UNDEFINED(radix) || radix == 10 || radix == 0) {
     // Some people use parseInt instead of Math.floor.  This
     // optimization makes parseInt on a Smi 12 times faster (60ns
     // vs 800ns).  The following optimization makes parseInt on a


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

Reply via email to