Title: [229476] trunk/Source/_javascript_Core
Revision
229476
Author
msab...@apple.com
Date
2018-03-09 12:27:40 -0800 (Fri, 09 Mar 2018)

Log Message

Relanding "testmasm crashes in testBranchTruncateDoubleToInt32() on ARM64"
https://bugs.webkit.org/show_bug.cgi?id=183488

It applied and built just fine locally.

* assembler/testmasm.cpp:
(JSC::testBranchTruncateDoubleToInt32):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (229475 => 229476)


--- trunk/Source/_javascript_Core/ChangeLog	2018-03-09 19:20:31 UTC (rev 229475)
+++ trunk/Source/_javascript_Core/ChangeLog	2018-03-09 20:27:40 UTC (rev 229476)
@@ -1,3 +1,13 @@
+2018-03-09  Michael Saboff  <msab...@apple.com>
+
+        Relanding "testmasm crashes in testBranchTruncateDoubleToInt32() on ARM64"
+        https://bugs.webkit.org/show_bug.cgi?id=183488
+
+        It applied and built just fine locally.
+
+        * assembler/testmasm.cpp:
+        (JSC::testBranchTruncateDoubleToInt32):
+
 2018-03-09  Yusuke Suzuki  <utatane....@gmail.com>
 
         Unreviewed, remove WebAssemblyFunctionType

Modified: trunk/Source/_javascript_Core/assembler/testmasm.cpp (229475 => 229476)


--- trunk/Source/_javascript_Core/assembler/testmasm.cpp	2018-03-09 19:20:31 UTC (rev 229475)
+++ trunk/Source/_javascript_Core/assembler/testmasm.cpp	2018-03-09 20:27:40 UTC (rev 229476)
@@ -32,6 +32,7 @@
 #include "InitializeThreading.h"
 #include "LinkBuffer.h"
 #include "ProbeContext.h"
+#include "StackAlignment.h"
 #include <limits>
 #include <wtf/Compiler.h>
 #include <wtf/DataLog.h>
@@ -198,7 +199,7 @@
 #endif
     CHECK_EQ(compileAndRun<int>([&] (CCallHelpers& jit) {
         jit.emitFunctionPrologue();
-        jit.subPtr(CCallHelpers::TrustedImm32(8), MacroAssembler::stackPointerRegister);
+        jit.subPtr(CCallHelpers::TrustedImm32(stackAlignmentBytes()), MacroAssembler::stackPointerRegister);
         if (isBigEndian) {
             jit.store32(CCallHelpers::TrustedImm32(valAsUInt >> 32),
                 MacroAssembler::stackPointerRegister);
@@ -218,7 +219,7 @@
         jit.move(CCallHelpers::TrustedImm32(0), GPRInfo::returnValueGPR);
 
         done.link(&jit);
-        jit.addPtr(CCallHelpers::TrustedImm32(8), MacroAssembler::stackPointerRegister);
+        jit.addPtr(CCallHelpers::TrustedImm32(stackAlignmentBytes()), MacroAssembler::stackPointerRegister);
         jit.emitFunctionEpilogue();
         jit.ret();
     }), expected);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to