Bug 1298466 - avoid some spammy warnings for ICU when compiling with clang-cl; r?ted.mielczarek draft
authorNathan Froyd <froydnj@mozilla.com>
Thu, 25 Aug 2016 17:13:51 -0400
changeset 406288 9634da995479dc1a4afad658fdeb88ce4c1aab47
parent 406287 bcea7b71027d61dd22309b81d4ce351b70ab2729
child 406289 790bd9dc4894982a38e34dd2a8c7f5f77c78815f
child 406780 8df77f79865ff263ae4948146a4b710f91c32f27
push id27678
push userbmo:nfroyd@mozilla.com
push dateFri, 26 Aug 2016 19:10:01 +0000
reviewersted.mielczarek
bugs1298466
milestone51.0a1
Bug 1298466 - avoid some spammy warnings for ICU when compiling with clang-cl; r?ted.mielczarek We don't need to hear about ICU redefining macros we set up or about include paths that are Microsoft-specific, but still resolve to where they need to go. MozReview-Commit-ID: D9lFiNCvkOh
config/external/icu/defs.mozbuild
--- a/config/external/icu/defs.mozbuild
+++ b/config/external/icu/defs.mozbuild
@@ -31,10 +31,22 @@ if CONFIG['MOZ_DEBUG']:
 if CONFIG['GNU_CXX']:
     CXXFLAGS += ['-frtti']
 elif CONFIG['OS_TARGET'] == 'WINNT':
     CXXFLAGS += ['-GR']
 
 DISABLE_STL_WRAPPING = True
 ALLOW_COMPILER_WARNINGS = True
 
+# We allow compiler warnings, but we can at least cut down on spammy
+# warnings that get triggered for every file.
+if CONFIG['CLANG_CL']:
+    CFLAGS += [
+        '-Wno-macro-redefined',
+        '-Wno-microsoft-include',
+    ]
+    CXXFLAGS += [
+        '-Wno-macro-redefined',
+        '-Wno-microsoft-include',
+    ]
+
 if CONFIG['OS_TARGET'] == 'Android' and CONFIG['MOZ_ANDROID_CXX_STL'] == 'mozstlport':
     LOCAL_INCLUDES += ['/build/gabi++/include']