Changeset 27615 in webkit for trunk/JavaScriptCore/kjs/nodes.cpp

Timestamp:

Nov 8, 2007, 2:23:39 PM (18 years ago)

Author:

oliver

Message:

Add a fast path for bitwise-and of two immediate numbers for a 0.7% improvement in SunSpider (4% bitop improvement).

Reviewed by Sam.

This only improves bitwise-and performance, as the additional logic required
for similar code paths on or, xor, and shifting requires additional operations
and branches that negate (and in certain cases, regress) any advantage we might
otherwise receive.

This improves performance on all bitop tests, the cryptography tests, as well as
the string-base64 and string-unpack-code tests. No significant degradation on
any other tests.

File:

• trunk/JavaScriptCore/kjs/nodes.cpp (modified) (2 diffs)

trunk/JavaScriptCore/kjs/nodes.cpp

@@ -2186 +2186 @@
 // ECMA 11.10
 JSValue *BitAndNode::evaluate(ExecState *exec)
-{
+{    
   JSValue *v1 = expr1->evaluate(exec);
   KJS_CHECKEXCEPTIONVALUE
@@ -2192 +2192 @@
   KJS_CHECKEXCEPTIONVALUE
   
-  return jsNumber(v1->toInt32(exec) & v2->toInt32(exec));
+  return jsNumberFromAnd(exec, v1, v2);
 }