Changeset 39879 in webkit for trunk/JavaScriptCore/runtime/JSImmediate.h

Timestamp:

Jan 13, 2009, 4:15:36 PM (16 years ago)

Author:

[email protected]

Message:

2009-01-13 Gavin Barraclough <​[email protected]>

Reviewed by Oliver Hunt.

Fix for: ​https://bugs.webkit.org/show_bug.cgi?id=23292

Implementation of two argument canDoFastAdditiveOperations does not correlate well with reality.

• runtime/JSImmediate.h: (JSC::JSFastMath::canDoFastAdditiveOperations):

File:

• trunk/JavaScriptCore/runtime/JSImmediate.h (modified) (1 diff)

trunk/JavaScriptCore/runtime/JSImmediate.h

@@ -699 +699 @@
             // Number is non-negative and an operation involving two of these can't overflow.
             // Checking for allowed negative numbers takes more time than it's worth on SunSpider.
-            return ((JSImmediate::rawValue(v1) | JSImmediate::rawValue(v2)) & (JSImmediate::TagTypeInteger + (JSImmediate::signBit | (JSImmediate::signBit >> 1)))) == JSImmediate::TagTypeInteger;
+            return canDoFastAdditiveOperations(v1) && canDoFastAdditiveOperations(v2);
         }