Changeset 59547 in webkit for trunk/JavaScriptCore/jit/JITArithmetic.cpp

Timestamp:

May 15, 2010, 12:22:34 PM (15 years ago)

Author:

[email protected]

Message:

2010-05-15 Oliver Hunt <​[email protected]>

Reviewed by Maciej Stachowiak.

Incorrect codegen for slowcase of < in 64-bit
​https://bugs.webkit.org/show_bug.cgi?id=39151

Call the correct stud for the slowcases of the < operator.

• jit/JITArithmetic.cpp: (JSC::JIT::emitSlow_op_jnless): (JSC::JIT::emitSlow_op_jless):

2010-05-15 Oliver Hunt <​[email protected]>

Reviewed by Maciej Stachowiak.

Incorrect codegen for slowcase of < in 64-bit
​https://bugs.webkit.org/show_bug.cgi?id=39151

Add yet more tests for the < operator when comparing integer strings.

• fast/js/comparison-operators-greater-expected.txt:
• fast/js/comparison-operators-less-expected.txt:
• fast/js/script-tests/comparison-operators-greater.js:
• fast/js/script-tests/comparison-operators-less.js:

File:

• trunk/JavaScriptCore/jit/JITArithmetic.cpp (modified) (2 diffs)

trunk/JavaScriptCore/jit/JITArithmetic.cpp

@@ -371 +371 @@
         linkSlowCase(iter);
         linkSlowCase(iter);
-        JITStubCall stubCall(this, cti_op_jlesseq);
+        JITStubCall stubCall(this, cti_op_jless);
         stubCall.addArgument(op1, regT0);
         stubCall.addArgument(op2, regT1);
@@ -586 +586 @@
         linkSlowCase(iter);
         linkSlowCase(iter);
-        JITStubCall stubCall(this, cti_op_jlesseq);
+        JITStubCall stubCall(this, cti_op_jless);
         stubCall.addArgument(op1, regT0);
         stubCall.addArgument(op2, regT1);