Title: [110877] trunk/Source/_javascript_Core
Revision
110877
Author
[email protected]
Date
2012-03-15 13:06:11 -0700 (Thu, 15 Mar 2012)

Log Message

ARMv7: prefer vmov(gpr,gpr->double) over vmov(gpr->single)
https://bugs.webkit.org/show_bug.cgi?id=81256

Reviewed by Oliver Hunt.

This is a 0.5% sunspider progression.

* assembler/MacroAssemblerARMv7.h:
(JSC::MacroAssemblerARMv7::convertInt32ToDouble):
    - switch which form of vmov we use.

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (110876 => 110877)


--- trunk/Source/_javascript_Core/ChangeLog	2012-03-15 19:55:37 UTC (rev 110876)
+++ trunk/Source/_javascript_Core/ChangeLog	2012-03-15 20:06:11 UTC (rev 110877)
@@ -1,3 +1,16 @@
+2012-03-15  Gavin Barraclough  <[email protected]>
+
+        ARMv7: prefer vmov(gpr,gpr->double) over vmov(gpr->single)
+        https://bugs.webkit.org/show_bug.cgi?id=81256
+
+        Reviewed by Oliver Hunt.
+
+        This is a 0.5% sunspider progression.
+
+        * assembler/MacroAssemblerARMv7.h:
+        (JSC::MacroAssemblerARMv7::convertInt32ToDouble):
+            - switch which form of vmov we use.
+
 2012-03-15  YoungTaeck Song  <[email protected]>
 
         [EFL] Add OwnPtr specialization for Ecore_Timer.

Modified: trunk/Source/_javascript_Core/assembler/MacroAssemblerARMv7.h (110876 => 110877)


--- trunk/Source/_javascript_Core/assembler/MacroAssemblerARMv7.h	2012-03-15 19:55:37 UTC (rev 110876)
+++ trunk/Source/_javascript_Core/assembler/MacroAssemblerARMv7.h	2012-03-15 20:06:11 UTC (rev 110877)
@@ -907,7 +907,7 @@
 
     void convertInt32ToDouble(RegisterID src, FPRegisterID dest)
     {
-        m_assembler.vmov(fpTempRegisterAsSingle(), src);
+        m_assembler.vmov(fpTempRegister, src, src);
         m_assembler.vcvt_signedToFloatingPoint(dest, fpTempRegisterAsSingle());
     }
 
@@ -915,7 +915,7 @@
     {
         // Fixme: load directly into the fpr!
         load32(address, dataTempRegister);
-        m_assembler.vmov(fpTempRegisterAsSingle(), dataTempRegister);
+        m_assembler.vmov(fpTempRegister, dataTempRegister, dataTempRegister);
         m_assembler.vcvt_signedToFloatingPoint(dest, fpTempRegisterAsSingle());
     }
 
@@ -923,7 +923,7 @@
     {
         // Fixme: load directly into the fpr!
         load32(address.m_ptr, dataTempRegister);
-        m_assembler.vmov(fpTempRegisterAsSingle(), dataTempRegister);
+        m_assembler.vmov(fpTempRegister, dataTempRegister, dataTempRegister);
         m_assembler.vcvt_signedToFloatingPoint(dest, fpTempRegisterAsSingle());
     }
     
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to