Title: [128815] trunk/Source/_javascript_Core
Revision
128815
Author
[email protected]
Date
2012-09-17 15:45:47 -0700 (Mon, 17 Sep 2012)

Log Message

Unreviewed, fix a broken assertion in offlineasm.

* offlineasm/armv7.rb:
* offlineasm/backends.rb:

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (128814 => 128815)


--- trunk/Source/_javascript_Core/ChangeLog	2012-09-17 22:45:05 UTC (rev 128814)
+++ trunk/Source/_javascript_Core/ChangeLog	2012-09-17 22:45:47 UTC (rev 128815)
@@ -1,3 +1,10 @@
+2012-09-17  Filip Pizlo  <[email protected]>
+
+        Unreviewed, fix a broken assertion in offlineasm.
+
+        * offlineasm/armv7.rb:
+        * offlineasm/backends.rb:
+
 2012-09-16  Mark Hahnenberg  <[email protected]>
 
         Delayed structure sweep can leak structures without bound

Modified: trunk/Source/_javascript_Core/offlineasm/armv7.rb (128814 => 128815)


--- trunk/Source/_javascript_Core/offlineasm/armv7.rb	2012-09-17 22:45:05 UTC (rev 128814)
+++ trunk/Source/_javascript_Core/offlineasm/armv7.rb	2012-09-17 22:45:47 UTC (rev 128815)
@@ -683,7 +683,7 @@
         $asm.puts "#{opcode3} #{armV7FlippedOperands(operands)}"
     else
         raise unless operands.size == 2
-        raise unless operands[1].is_a? RegisterID
+        raise unless operands[1].register?
         if operands[0].is_a? Immediate
             $asm.puts "#{opcode3} #{operands[1].armV7Operand}, #{operands[1].armV7Operand}, #{operands[0].armV7Operand}"
         else

Modified: trunk/Source/_javascript_Core/offlineasm/backends.rb (128814 => 128815)


--- trunk/Source/_javascript_Core/offlineasm/backends.rb	2012-09-17 22:45:05 UTC (rev 128814)
+++ trunk/Source/_javascript_Core/offlineasm/backends.rb	2012-09-17 22:45:47 UTC (rev 128815)
@@ -56,7 +56,8 @@
             $activeBackend = name
             send("lower" + name)
         rescue => e
-            raise "Got error #{e} at #{codeOriginString}"
+            e.message << "At #{codeOriginString}"
+            raise e
         end
     end
 end
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to