Preprocessing text files with modern clang treats double slashes in comments
authorap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 13 Apr 2012 01:54:08 +0000 (01:54 +0000)
committerap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 13 Apr 2012 01:54:08 +0000 (01:54 +0000)
https://bugs.webkit.org/show_bug.cgi?id=83827

Reviewed by Mark Rowe.

Source/WebCore:

* DerivedSources.make: Pass appropriate flags to both llvm and gcc.

Source/WebKit2:

* DerivedSources.make: Pass appropriate flags to both llvm and gcc.

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

Source/WebCore/ChangeLog
Source/WebCore/DerivedSources.make
Source/WebKit2/ChangeLog
Source/WebKit2/DerivedSources.make

index 6bc9dfc..72e887b 100644 (file)
@@ -1,3 +1,12 @@
+2012-04-12  Alexey Proskuryakov  <ap@apple.com>
+
+        Preprocessing text files with modern clang treats double slashes in comments
+        https://bugs.webkit.org/show_bug.cgi?id=83827
+
+        Reviewed by Mark Rowe.
+
+        * DerivedSources.make: Pass appropriate flags to both llvm and gcc.
+
 2012-04-12  David Dorwin  <ddorwin@chromium.org>
 
         Add Encrypted Media Extensions events and errors to HTMLMediaElement
index d888ab8..d158343 100644 (file)
@@ -647,6 +647,12 @@ ifeq ($(OS),MACOS)
 FRAMEWORK_FLAGS = $(shell echo $(BUILT_PRODUCTS_DIR) $(FRAMEWORK_SEARCH_PATHS) | perl -e 'print "-F " . join(" -F ", split(" ", <>));')
 HEADER_FLAGS = $(shell echo $(BUILT_PRODUCTS_DIR) $(HEADER_SEARCH_PATHS) | perl -e 'print "-I" . join(" -I", split(" ", <>));')
 
+ifeq ($(TARGET_GCC_VERSION),LLVM_COMPILER)
+       TEXT_PREPROCESSOR_FLAGS=-E -P -x c -traditional
+else
+       TEXT_PREPROCESSOR_FLAGS=-E -P -x c -std=c89
+endif
+
 ifeq ($(shell $(CC) -x c++ -E -P -dM $(FRAMEWORK_FLAGS) $(HEADER_FLAGS) -include "wtf/Platform.h" /dev/null | grep ENABLE_DASHBOARD_SUPPORT | cut -d' ' -f3), 1)
     ENABLE_DASHBOARD_SUPPORT = 1
 else
@@ -663,7 +669,7 @@ all: DefaultFonts.plist
 
 DefaultFonts.plist : DefaultFonts.plist.in
        @echo Pre-processing DefaultFonts.plist...
-       $(CC) -E -P -w -x c -std=c89 $(FRAMEWORK_FLAGS) $(HEADER_FLAGS) -include "wtf/Platform.h" $< > $@
+       $(CC) $(TEXT_PREPROCESSOR_FLAGS) $(FRAMEWORK_FLAGS) $(HEADER_FLAGS) -include "wtf/Platform.h" $< > $@
 
 else
 
index d2d8bfe..e1a532a 100644 (file)
@@ -1,3 +1,12 @@
+2012-04-12  Alexey Proskuryakov  <ap@apple.com>
+
+        Preprocessing text files with modern clang treats double slashes in comments
+        https://bugs.webkit.org/show_bug.cgi?id=83827
+
+        Reviewed by Mark Rowe.
+
+        * DerivedSources.make: Pass appropriate flags to both llvm and gcc.
+
 2012-04-12  Brent Fulgham  <bfulgham@webkit.org>
 
         [WinCairo] Build fix after Windows export definition file change.
index f1101fb..ddb8e9a 100644 (file)
@@ -118,6 +118,12 @@ ifeq ($(OS),MACOS)
 FRAMEWORK_FLAGS = $(shell echo $(BUILT_PRODUCTS_DIR) $(FRAMEWORK_SEARCH_PATHS) | perl -e 'print "-F " . join(" -F ", split(" ", <>));')
 HEADER_FLAGS = $(shell echo $(BUILT_PRODUCTS_DIR) $(HEADER_SEARCH_PATHS) | perl -e 'print "-I" . join(" -I", split(" ", <>));')
 
+ifeq ($(TARGET_GCC_VERSION),LLVM_COMPILER)
+       TEXT_PREPROCESSOR_FLAGS=-E -P -x c -traditional
+else
+       TEXT_PREPROCESSOR_FLAGS=-E -P -x c -std=c89
+endif
+
 SANDBOX_PROFILES = \
        com.apple.WebProcess.sb \
        com.apple.WebKit.PluginProcess.sb
@@ -126,7 +132,7 @@ all: $(SANDBOX_PROFILES)
 
 %.sb : %.sb.in
        @echo Pre-processing $* sandbox profile...
-       $(CC) -E -P -w -x c -std=c89 $(FRAMEWORK_FLAGS) $(HEADER_FLAGS) -include "wtf/Platform.h" $< > $@
+       $(CC) $(TEXT_PREPROCESSOR_FLAGS) $(FRAMEWORK_FLAGS) $(HEADER_FLAGS) -include "wtf/Platform.h" $< > $@
 
 endif # MACOS