Changeset 36746 in webkit for trunk/JavaScriptCore/VM/Machine.cpp

Timestamp:

Sep 21, 2008, 4:08:15 PM (17 years ago)

Author:

Darin Adler

Message:

2008-09-21 Darin Adler <Darin Adler>

Reviewed by Maciej Stachowiak.

• ​https://bugs.webkit.org/show_bug.cgi?id=20975 inline immediate-number case of ==

• VM/CTI.h: Renamed emitJumpSlowCaseIfNotImm to emitJumpSlowCaseIfNotImmNum, since the old name was incorrect.

• VM/CTI.cpp: Updated for new name. (JSC::CTI::privateCompileMainPass): Added op_eq. (JSC::CTI::privateCompileSlowCases): Added op_eq.

• VM/Machine.cpp: (JSC::Machine::cti_op_eq): Removed fast case, since it's now compiled.

File:

• trunk/JavaScriptCore/VM/Machine.cpp (modified) (1 diff)

trunk/JavaScriptCore/VM/Machine.cpp

@@ -4951 +4951 @@
     JSValue* src2 = ARG_src2;
 
-    if (JSImmediate::areBothImmediateNumbers(src1, src2))
-        return jsBoolean(reinterpret_cast<intptr_t>(src1) == reinterpret_cast<intptr_t>(src2));
-    else {
-        ExecState* exec = ARG_exec;
-        JSValue* result = jsBoolean(equal(exec, src1, src2));
-        VM_CHECK_EXCEPTION_AT_END();
-        return result;
-    }
+    ExecState* exec = ARG_exec;
+
+    ASSERT(!JSImmediate::areBothImmediateNumbers(src1, src2));
+    JSValue* result = jsBoolean(equal(exec, src1, src2));
+    VM_CHECK_EXCEPTION_AT_END();
+    return result;
 }