Changeset 73981 in webkit

Timestamp:

Dec 13, 2010 4:02:49 PM (13 years ago)

Author:

cwzwarich@webkit.org

Message:

Reviewed by Gavin Barraclough.

JavaScriptCore should not use "asm volatile" outside of a function
​https://bugs.webkit.org/show_bug.cgi?id=50991

• jit/JITStubs.cpp: Remove the volatile keyword from asm statements.

Location:

trunk/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• jit/JITStubs.cpp (modified) (10 diffs)

trunk/JavaScriptCore/ChangeLog

@@ +1 @@
+2010-12-13  Cameron Zwarich  <zwarich@apple.com>
+
+        Reviewed by Gavin Barraclough.
+
+        JavaScriptCore should not use "asm volatile" outside of a function
+        https://bugs.webkit.org/show_bug.cgi?id=50991
+
+        * jit/JITStubs.cpp: Remove the volatile keyword from asm statements.
+
 2010-12-13  Steve Falkenburg  <sfalken@apple.com>
 

trunk/JavaScriptCore/jit/JITStubs.cpp

@@ -547 +547 @@
 
 #if CPU(MIPS)
-asm volatile(
+asm (
 ".text" "\n"
 ".align 2" "\n"
@@ -586 +586 @@
 );
 
-asm volatile(
+asm (
 ".text" "\n"
 ".align 2" "\n"
@@ -617 +617 @@
 );
 
-asm volatile(
+asm (
 ".text" "\n"
 ".align 2" "\n"
@@ -640 +640 @@
 #if COMPILER(GCC) && CPU(ARM_THUMB2)
 
-asm volatile(
+asm (
 ".text" "\n"
 ".align 2" "\n"
@@ -667 +667 @@
 );
 
-asm volatile(
+asm (
 ".text" "\n"
 ".align 2" "\n"
@@ -685 +685 @@
 );
 
-asm volatile(
+asm (
 ".text" "\n"
 ".align 2" "\n"
@@ -703 +703 @@
 #elif COMPILER(GCC) && CPU(ARM_TRADITIONAL)
 
-asm volatile(
+asm (
 ".globl " SYMBOL_STRING(ctiTrampoline) "\n"
 HIDE_SYMBOL(ctiTrampoline) "\n"
@@ -721 +721 @@
 );
 
-asm volatile(
+asm (
 ".globl " SYMBOL_STRING(ctiVMThrowTrampoline) "\n"
 HIDE_SYMBOL(ctiVMThrowTrampoline) "\n"
@@ -1109 +1109 @@
         rtype JITStubThunked_##op(STUB_ARGS_DECLARATION); \
     }; \
-    asm volatile( \
+    asm ( \
         ".text" "\n" \
         ".align 2" "\n" \
@@ -1139 +1139 @@
         rtype JITStubThunked_##op(STUB_ARGS_DECLARATION); \
     }; \
-    asm volatile( \
+    asm ( \
         ".text" "\n" \
         ".align 2" "\n" \