Fix http://bugs.webkit.org/show_bug.cgi?id=17175 (Bug 17175: Use of C++ compiler...
authormrowe@apple.com <mrowe@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 5 Feb 2008 03:25:59 +0000 (03:25 +0000)
committermrowe@apple.com <mrowe@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 5 Feb 2008 03:25:59 +0000 (03:25 +0000)
Reviewed by Alp Toker and Mark Rowe.

Add global_cxxflags definition for inclusion in CXXFLAGS variables.
Only use -fno-rtti and $(SYMBOL_VISIBILITY_INLINES) with global_cxxflags as gcc complains they aren't valid for C.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29986 268f45cc-cd09-0410-ab3c-d52691b4dbfc

ChangeLog
GNUmakefile.am
JavaScriptCore/ChangeLog
JavaScriptCore/GNUmakefile.am
WebKitTools/ChangeLog
WebKitTools/GNUmakefile.am

index a949e19..7865d28 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,15 @@
+2008-02-04  Rodney Dawes  <dobey@wayofthemonkey.com>
+
+        Reviewed by Alp Toker and Mark Rowe.
+
+        Fix http://bugs.webkit.org/show_bug.cgi?id=17175.
+        Bug 17175: Use of C++ compiler flags in CFLAGS
+
+        Add global_cxxflags definition for inclusion in CXXFLAGS variables.
+        Only use -fno-rtti and $(SYMBOL_VISIBILITY_INLINES) with global_cxxflags as gcc complains they aren't valid for C.
+
+        * GNUmakefile.am:
+
 2008-02-04  Alp Toker  <alp@atoker.com>
 
         Rubber-stamped by Mark Rowe.
index 2cc5533..054265a 100644 (file)
 #
 # global_cppflags   = CPPFLAGS that apply to JSC, WebCore, and to any
 #                     specific port
-# global_cflags     = CFLAGS and CXXFLAGS that apply to JSC, WebCore, and to
+# global_cflags     = CFLAGS that apply to JSC, WebCore, and to
 #                     any specific port
+# global_cxxflags   = CXXFLAGS that apply to JSC, WebCore, and to any
+#                     specific port
 
 srcdir = @srcdir@
 VPATH = @srcdir@
@@ -61,12 +63,18 @@ global_cflags := \
        -Wall -W -Wcast-align -Wchar-subscripts -Wreturn-type \
        -Wformat -Wformat-security -Wno-format-y2k -Wundef \
        -Wmissing-format-attribute -Wpointer-arith -Wwrite-strings \
-       -Wno-unused-parameter $(SYMBOL_VISIBILITY_INLINES)
+       -Wno-unused-parameter
+
+global_cxxflags := \
+       $(SYMBOL_VISIBILITY_INLINES)
 
 if !ENABLE_DEBUG
 global_cflags += \
-       -fno-rtti -fno-exceptions \
+       -fno-exceptions \
        $(SYMBOL_VISIBILITY)
+
+global_cxxflags += \
+       -fno-rtti
 endif
 
 # Shared libraries
@@ -96,7 +104,7 @@ libJavaScriptCore_la_SOURCES = \
 
 libJavaScriptCore_la_LDFLAGS = $(ICU_LIBS) -lpthread
 
-libJavaScriptCore_la_CXXFLAGS = $(global_cflags) -fstrict-aliasing
+libJavaScriptCore_la_CXXFLAGS = $(global_cxxflags) $(global_cflags) -fstrict-aliasing
 
 libJavaScriptCore_la_CFLAGS = $(global_cflags) -fstrict-aliasing
 
@@ -139,6 +147,7 @@ libWebKitGtk_la_SOURCES = \
 
 libWebKitGtk_la_CXXFLAGS = \
        -fno-strict-aliasing \
+       $(global_cxxflags) \
        $(global_cflags) \
        $(DEPENDENCIES_CFLAGS) \
        $(LIBCURL_CFLAGS) \
index 09eae74..ad4edf7 100644 (file)
@@ -1,3 +1,12 @@
+2008-02-04  Rodney Dawes  <dobey@wayofthemonkey.com>
+
+        Reviewed by Alp Toker and Mark Rowe.
+
+        Fix http://bugs.webkit.org/show_bug.cgi?id=17175.
+        Bug 17175: Use of C++ compiler flags in CFLAGS
+
+        * GNUmakefile.am: Use global_cxxflags as well as global_cflags in CXXFLAGS.
+
 2008-02-04  Alp Toker  <alp@atoker.com>
 
         Rubber-stamped by Mark Rowe.
@@ -17,7 +26,7 @@
         Geoff's theory is that the slowdown was due to copying hash tables when
         putting things into the back/forward cache. If that's true, then this
         should fix the problem.
-        
+
         (According to Geoff's measurements, in a PLT that exaggerates the
         importance of symbol table saving during cached page creation, this
         patch is a ~3X speedup in cached page creation, and a 9% speedup overall.)
index fb6e143..b1f0274 100644 (file)
@@ -149,14 +149,14 @@ Programs_minidom_SOURCES = \
        JavaScriptCore/API/NodeList.c \
        JavaScriptCore/API/minidom.c
 Programs_minidom_CPPFLAGS = $(global_cppflags)
-Programs_minidom_CXXFLAGS = $(global_cflags)
+Programs_minidom_CXXFLAGS = $(global_cxxflags) $(global_cflags)
 Programs_minidom_LDADD = libJavaScriptCore.la
 Programs_minidom_LDFLAGS = -rpath $(CURDIR)/.libs
 
 # testkjs
 Programs_testkjs_SOURCES = JavaScriptCore/kjs/testkjs.cpp
 Programs_testkjs_CPPFLAGS = $(global_cppflags)
-Programs_testkjs_CXXFLAGS = $(global_cflags)
+Programs_testkjs_CXXFLAGS = $(global_cxxflags) $(global_cflags)
 Programs_testkjs_LDADD = libJavaScriptCore.la
 Programs_testkjs_LDFLAGS = -rpath $(CURDIR)/.libs
 
index a00cd0e..9bff36c 100644 (file)
@@ -1,3 +1,12 @@
+2008-02-04  Rodney Dawes  <dobey@wayofthemonkey.com>
+
+        Reviewed by Alp Toker and Mark Rowe.
+
+        Fix http://bugs.webkit.org/show_bug.cgi?id=17175.
+        Bug 17175: Use of C++ compiler flags in CFLAGS
+
+        * GNUmakefile.am: Use global_cxxflags as well as global_cflags in CXXFLAGS.
+
 2008-02-04  Darin Adler  <darin@apple.com>
 
         Reviewed by Geoff.
index e7b049c..e42a064 100644 (file)
@@ -39,6 +39,7 @@ Programs_DumpRenderTree_SOURCES = \
 
 Programs_DumpRenderTree_CXXFLAGS = \
        $(DEPENDENCIES_CFLAGS) \
+       $(global_cxxflags) \
        $(global_cflags)
 
 Programs_DumpRenderTree_LDADD = \