Changeset 145560 in webkit

Timestamp:

Mar 12, 2013 10:40:40 AM (11 years ago)

Author:

zandobersek@gmail.com

Message:

[GTK] 'unrecognized command line option "-Wno-c++11-extensions"' warning reported when using gcc
​https://bugs.webkit.org/show_bug.cgi?id=111936

Reviewed by Martin Robinson.

• Source/autotools/SetupCompilerFlags.m4: Only use the -Wno-c++11-extensions warning when building

with Clang as GCC does not support the warning just yet. Also enhance the comment about using libstdc++
as the standard C++ library when compiling with Clang.

Location:

trunk

Files:

• ChangeLog (modified) (1 diff)
• Source/autotools/SetupCompilerFlags.m4 (modified) (2 diffs)

trunk/ChangeLog

@@ +1 @@
+2013-03-12  Zan Dobersek  <zdobersek@igalia.com>
+
+        [GTK] 'unrecognized command line option "-Wno-c++11-extensions"' warning reported when using gcc
+        https://bugs.webkit.org/show_bug.cgi?id=111936
+
+        Reviewed by Martin Robinson.
+
+        * Source/autotools/SetupCompilerFlags.m4: Only use the -Wno-c++11-extensions warning when building
+        with Clang as GCC does not support the warning just yet. Also enhance the comment about using libstdc++
+        as the standard C++ library when compiling with Clang.
+
 2013-03-12  Zan Dobersek  <zdobersek@igalia.com>
 

trunk/Source/autotools/SetupCompilerFlags.m4

@@ -1 +1 @@
 # Use C99 as the language standard for C code.
 CFLAGS="$CFLAGS -std=c99"
-# Do not warn about C++11 incompatibilities and extensions.
-CXXFLAGS="$CXXFLAGS -Wno-c++11-compat -Wno-c++11-extensions"
+# Do not warn about C++11 incompatibilities.
+CXXFLAGS="$CXXFLAGS -Wno-c++11-compat"
 
 # Clang requires suppression of unused arguments warning.
@@ -8 +8 @@
     CFLAGS="$CFLAGS -Qunused-arguments"
 fi
-# libstdc++ is at the moment the only option as the C++ standard library.
+# -Wno-c++11-extensions, currently only usable with Clang, suppresses warnings of C++11 extensions in use.
+# libstdc++ is at the moment the only option as the C++ standard library when compiling with Clang.
 if test "$CXX" = "clang++"; then
-    CXXFLAGS="$CXXFLAGS -stdlib=libstdc++ -Qunused-arguments"
+    CXXFLAGS="$CXXFLAGS -Wno-c++11-extensions -stdlib=libstdc++ -Qunused-arguments"
 fi