Changeset 37755 in webkit for trunk/JavaScriptCore

Timestamp:

Oct 20, 2008, 7:45:57 PM (17 years ago)

Author:

[email protected]

Message:

2008-10-20 Cameron Zwarich <​[email protected]>

Reviewed by Maciej Stachowiak.

Remove an untaken branch in CodeGenerator::emitJumpIfFalse(). This
function is never called with a backwards target LabelID, and there is
even an assertion to this effect at the top of the function body.

• VM/CodeGenerator.cpp: (JSC::CodeGenerator::emitJumpIfFalse):

Location:

trunk/JavaScriptCore

Files:

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

trunk/JavaScriptCore/ChangeLog

@@ +1 @@
+2008-10-20  Cameron Zwarich  <[email protected]>
+
+        Reviewed by Maciej Stachowiak.
+
+        Remove an untaken branch in CodeGenerator::emitJumpIfFalse(). This
+        function is never called with a backwards target LabelID, and there is
+        even an assertion to this effect at the top of the function body.
+
+        * VM/CodeGenerator.cpp:
+        (JSC::CodeGenerator::emitJumpIfFalse):
+
 2008-10-20  Cameron Zwarich  <[email protected]>
 

trunk/JavaScriptCore/VM/CodeGenerator.cpp

@@ -582 +582 @@
         if (cond->index() == dstIndex && cond->isTemporary() && !cond->refCount()) {
             rewindUnaryOp();
-            emitOpcode(target->isForwardLabel() ? op_jtrue : op_loop_if_true);
+            emitOpcode(op_jtrue);
             instructions().append(srcIndex);
             instructions().append(target->offsetFrom(instructions().size()));