Changeset 175544 in webkit

Timestamp:

Nov 4, 2014 6:32:03 AM (10 years ago)

Author:

Dániel Bátyai

Message:

ATTR_USED incorrectly applying to Clang
​https://bugs.webkit.org/show_bug.cgi?id=138313

Reviewed by Csaba Osztrogonác.

ATTR_USED was only intended for GCC. Also limit it to ENABLE(FTL_NATIVE_CALL_INLINING)
since that's the only thing it's needed for.

• wtf/Platform.h:

Location:

trunk/Source/WTF

Files:

• ChangeLog (modified) (1 diff)
• wtf/Platform.h (modified) (1 diff)

trunk/Source/WTF/ChangeLog

@@ +1 @@
+2014-11-04  Dániel Bátyai  <dbatyai.u-szeged@partner.samsung.com>
+
+        ATTR_USED incorrectly applying to Clang
+        https://bugs.webkit.org/show_bug.cgi?id=138313
+
+        Reviewed by Csaba Osztrogonác.
+
+        ATTR_USED was only intended for GCC. Also limit it to ENABLE(FTL_NATIVE_CALL_INLINING)
+        since that's the only thing it's needed for.
+
+        * wtf/Platform.h:
+
 2014-11-03  Chris Dumez  <cdumez@apple.com>
 

trunk/Source/WTF/wtf/Platform.h

@@ -748 +748 @@
 
 /* Used to make GCC's optimization not throw away a symbol that we would need for native inlining */
-#if COMPILER(GCC)
+#if ENABLE(FTL_NATIVE_CALL_INLINING) && COMPILER(GCC) && !COMPILER(CLANG)
 #define ATTR_USED __attribute__ ((used))
 #else