Changeset 37602 in webkit for trunk/JavaScriptCore

Timestamp:

Oct 15, 2008, 2:35:10 AM (17 years ago)

Author:

[email protected]

Message:

2008-10-15 Gabor Loki <​[email protected]>

Reviewed by Cameron Zwarich.

<​https://bugs.webkit.org/show_bug.cgi?id=20912>
Use simple uint32_t multiplication on op_mul if both operands are
immediate number and they are between zero and 0x7FFF.

• VM/Machine.cpp: (JSC::Machine::privateExecute):

Location:

trunk/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• VM/Machine.cpp (modified) (1 diff)

trunk/JavaScriptCore/ChangeLog

@@ +1 @@
+2008-10-15  Gabor Loki  <[email protected]>
+
+        Reviewed by Cameron Zwarich.
+
+        <https://bugs.webkit.org/show_bug.cgi?id=20912>
+        Use simple uint32_t multiplication on op_mul if both operands are
+        immediate number and they are between zero and 0x7FFF.
+
+        * VM/Machine.cpp:
+        (JSC::Machine::privateExecute):
+
 2008-10-09  Darin Fisher  <[email protected]>
 

trunk/JavaScriptCore/VM/Machine.cpp

@@ -1865 +1865 @@
         double left;
         double right;
-        if (fastIsNumber(src1, left) && fastIsNumber(src2, right))
+        if (JSImmediate::areBothImmediateNumbers(src1, src2)) {
+            int32_t left = JSImmediate::getTruncatedInt32(src1);
+            int32_t right = JSImmediate::getTruncatedInt32(src2);
+            if ((left | right) >> 15 == 0)
+                callFrame[dst] = jsNumber(callFrame, left * right);
+            else
+                callFrame[dst] = jsNumber(callFrame, static_cast<double>(left) * static_cast<double>(right));
+        } else if (fastIsNumber(src1, left) && fastIsNumber(src2, right))
             callFrame[dst] = jsNumber(callFrame, left * right);
         else {