Changeset 154095 in webkit for trunk/Source/JavaScriptCore/offlineasm/x86.rb

Timestamp:

Aug 15, 2013, 4:28:55 AM (12 years ago)

Author:

[email protected]

Message:

REGRESSION(r148790) Made 7 tests fail on x86 32bit
​https://bugs.webkit.org/show_bug.cgi?id=114913

Reviewed by Filip Pizlo.

Source/JavaScriptCore:

The X87 register was not freed before some calls. Instead
of inserting resetX87Registers to the last call sites,
the two X87 registers are now freed in every call.

• llint/LowLevelInterpreter32_64.asm:
• llint/LowLevelInterpreter64.asm:
• offlineasm/instructions.rb:
• offlineasm/x86.rb:

LayoutTests:

Unskip the five tests now passsing again.

• platform/qt/TestExpectations:

File:

• trunk/Source/JavaScriptCore/offlineasm/x86.rb (modified) (2 diffs)

trunk/Source/JavaScriptCore/offlineasm/x86.rb

@@ -1150 +1150 @@
             $asm.puts "int $3"
         when "call"
+            if useX87
+                2.times {
+                    | offset |
+                    $asm.puts "ffree %st(#{offset})"
+                }
+            end
             $asm.puts "call #{operands[0].x86CallOperand(:ptr)}"
         when "ret"
@@ -1337 +1343 @@
         when "leap"
             $asm.puts "lea#{x86Suffix(:ptr)} #{operands[0].x86AddressOperand(:ptr)}, #{operands[1].x86Operand(:ptr)}"
-        when "resetX87Stack"
-            if useX87
-                2.times {
-                    | offset |
-                    $asm.puts "ffree %st(#{offset})"
-                }
-            end
         else
             lowerDefault