Changeset 176100 in webkit

Timestamp:

Nov 13, 2014 3:43:46 PM (9 years ago)

Author:

mark.lam@apple.com

Message:

Change X86/64 JIT probes to save/restore xmm regs as double instead of m128. [Follow up]
<​https://webkit.org/b/138708>

Reviewed by Michael Saboff.

Removed a stale comment and a now unnecessary #include.

• assembler/X86Assembler.h:

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• assembler/X86Assembler.h (modified) (2 diffs)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2014-11-13  Mark Lam  <mark.lam@apple.com>
+
+        Change X86/64 JIT probes to save/restore xmm regs as double instead of __m128. [Follow up]
+        <https://webkit.org/b/138708>
+
+        Reviewed by Michael Saboff.
+
+        Removed a stale comment and a now unnecessary #include. 
+
+        * assembler/X86Assembler.h:
+
 2014-11-13  Commit Queue  <commit-queue@webkit.org>
 

trunk/Source/JavaScriptCore/assembler/X86Assembler.h

@@ -36 +36 @@
 #include <wtf/Vector.h>
 
-#if ENABLE(MASM_PROBE)
-#include <xmmintrin.h>
-#endif
-
 namespace JSC {
 
@@ -69 +65 @@
     V(void*, eflags) \
 
-// Note: the JITs only stores double values in the FP registers. However, the
-// storage type here is defined as __m128 because the JIT probe code which does
-// the storing still expects a __m128 slot. This will be changed when the JIT
-// probe code is updated later to reflect the JITs' usage of these registers.
+// Note: the JITs only stores double values in the FP registers.
 #define FOR_EACH_CPU_FPREGISTER(V) \
     V(double, xmm0) \