Title: [161431] branches/jsCStack/Source/_javascript_Core
Revision
161431
Author
msab...@apple.com
Date
2014-01-07 09:57:17 -0800 (Tue, 07 Jan 2014)

Log Message

CStack Branch: Disable JIT stack alignment check for ARM64
https://bugs.webkit.org/show_bug.cgi?id=126550

Unreviewed comment only change.

Added comment to checkStackPointerAlignment() suggested by Darin Adler.

Modified Paths

Diff

Modified: branches/jsCStack/Source/_javascript_Core/ChangeLog (161430 => 161431)


--- branches/jsCStack/Source/_javascript_Core/ChangeLog	2014-01-07 17:49:04 UTC (rev 161430)
+++ branches/jsCStack/Source/_javascript_Core/ChangeLog	2014-01-07 17:57:17 UTC (rev 161431)
@@ -1,3 +1,12 @@
+2014-01-07  Michael Saboff  <msab...@apple.com>
+
+        CStack Branch: Disable JIT stack alignment check for ARM64
+        https://bugs.webkit.org/show_bug.cgi?id=126550
+
+        Unreviewed comment only change.
+
+        Added comment to checkStackPointerAlignment() suggested by Darin Adler.
+
 2014-01-06  Filip Pizlo  <fpi...@apple.com>
 
         Merge trunk r161072, r161126, r161218, r161353, r161356, r161364, 161399.

Modified: branches/jsCStack/Source/_javascript_Core/jit/AssemblyHelpers.h (161430 => 161431)


--- branches/jsCStack/Source/_javascript_Core/jit/AssemblyHelpers.h	2014-01-07 17:49:04 UTC (rev 161430)
+++ branches/jsCStack/Source/_javascript_Core/jit/AssemblyHelpers.h	2014-01-07 17:57:17 UTC (rev 161431)
@@ -61,6 +61,7 @@
 
     void checkStackPointerAlignment()
     {
+        // This check is both unneeded and harder to write correctly for ARM64
 #if !defined(NDEBUG) && !CPU(ARM64)
         Jump stackPointerAligned = branchTestPtr(Zero, stackPointerRegister, TrustedImm32(0xf));
         breakpoint();
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to