Changeset 157473 in webkit

Timestamp:

Oct 15, 2013, 3:05:02 PM (12 years ago)

Author:

mark.lam@apple.com

Message:

Fix 3 operand sub operation in C loop LLINT.
​https://bugs.webkit.org/show_bug.cgi?id=122866.

Reviewed by Geoffrey Garen.

• offlineasm/cloop.rb:

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• offlineasm/cloop.rb (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2013-10-15  Mark Lam  <mark.lam@apple.com>
+
+        Fix 3 operand sub operation in C loop LLINT.
+        https://bugs.webkit.org/show_bug.cgi?id=122866.
+
+        Reviewed by Geoffrey Garen.
+
+        * offlineasm/cloop.rb:
+
 2013-10-15  Mark Hahnenberg  <mhahnenberg@apple.com>
 

trunk/Source/JavaScriptCore/offlineasm/cloop.rb

@@ -357 +357 @@
         type == :int64 || type == :uint64 || type == :double
     if operands.size == 3
-        $asm.putc "#{operands[2].clValue(type)} = #{operands[1].clValue(type)} #{operator} #{operands[0].clValue(type)};"
+        $asm.putc "#{operands[2].clValue(type)} = #{operands[0].clValue(type)} #{operator} #{operands[1].clValue(type)};"
         if operands[2].is_a? RegisterID and (type == :int32 or type == :uint32)
             $asm.putc "#{operands[2].clDump}.clearHighWord();" # Just clear it. It does nothing on the 32-bit port.