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

Timestamp:

Aug 17, 2008, 2:27:56 PM (17 years ago)

Author:

[email protected]

Message:

2008-08-17 Cameron Zwarich <​[email protected]>

Reviewed by Maciej.

Bug 20419: Remove op_jless
<​https://bugs.webkit.org/show_bug.cgi?id=20419>

Remove op_jless, which is rarely used now that we have op_loop_if_less.

• VM/CodeBlock.cpp: (KJS::CodeBlock::dump):
• VM/CodeGenerator.cpp: (KJS::CodeGenerator::emitJumpIfTrue):
• VM/Machine.cpp: (KJS::Machine::privateExecute):
• VM/Opcode.h:

File:

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

trunk/JavaScriptCore/VM/Machine.cpp

@@ -2215 +2215 @@
         NEXT_OPCODE;
     }
-    BEGIN_OPCODE(op_jless) {
-        /* jless src1(r) src2(r) target(offset)
-
-           Checks whether register src1 is less than register src2, as
-           with the ECMAScript '<' operator, and then jumps to offset
-           target from the current instruction, if and only if the 
-           result of the comparison is true.
-        */
-        JSValue* src1 = r[(++vPC)->u.operand].jsValue(exec);
-        JSValue* src2 = r[(++vPC)->u.operand].jsValue(exec);
-        int target = (++vPC)->u.operand;
-
-        bool result = jsLess(exec, src1, src2);
-        VM_CHECK_EXCEPTION();
-        
-        if (result) {
-            vPC += target;
-            NEXT_OPCODE;
-        }
-
-        ++vPC;
-        NEXT_OPCODE;
-    }
     BEGIN_OPCODE(op_jnless) {
         /* jnless src1(r) src2(r) target(offset)