Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4)

Unified Diff: third_party/mesa/BUILD.gn

Issue 1151303006: clang/win: Make everything work in a GN build. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: nacl fix Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build/config/compiler/BUILD.gn ('k') | tools/gn/docs/cookbook.md » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/mesa/BUILD.gn
diff --git a/third_party/mesa/BUILD.gn b/third_party/mesa/BUILD.gn
index acdca8616273962c6b81f55e89cd44e26f4465cd..da9d1ac57ea5e939b3c39e0fe09f0a52390fe1cb 100644
--- a/third_party/mesa/BUILD.gn
+++ b/third_party/mesa/BUILD.gn
@@ -78,10 +78,6 @@ config("mesa_internal_config") {
"$generated_src_dir/mesa/glapi",
]
- if (is_clang) {
- cflags += [ "-Wno-tautological-constant-out-of-range-compare" ]
- }
-
if (is_android) {
defines += [
"__GLIBC__",
@@ -108,7 +104,20 @@ config("mesa_internal_config") {
cflags += [ "-fPIC" ]
}
}
+}
+# mesa_internal_config is prepended to the config lists. Flags that
+# disable warnings need to be appended instead so that they show up
+# after flags like -Wall. (gn orders flags on a target before flags from
+# configs.)
+config("mesa_internal_warnings") {
+ cflags = []
+ if (is_clang) {
+ cflags += [
+ "-Wno-tautological-constant-out-of-range-compare",
+ "-Wno-mismatched-tags", # Fixed upstream.
+ ]
+ }
if (is_win) {
# TODO(scottmg): http://crbug.com/143877 These should be removed if
# Mesa is ever rolled and the warnings are fixed.
@@ -117,7 +126,6 @@ config("mesa_internal_config") {
"/wd4065", # Switch statement contains 'default' but no 'case' labels.
"/wd4090", # 'Operation' : different 'modifier' qualifiers
"/wd4099", # Type name struct-vs-class doesn't match.
- "/wd4267", # size_t to type.
"/wd4273", # Inconsistent DLL linkage.
"/wd4291", # No matching operator delete found for placement new.
"/wd4305", # Truncation from int to float.
@@ -235,7 +243,8 @@ static_library("mesa_libglslcommon") {
configs += [ "//build/config/compiler:no_chromium_code" ]
previous_configs = configs
configs = []
- configs = [ ":mesa_internal_config" ] + previous_configs
+ configs = [ ":mesa_internal_config" ] + previous_configs +
+ [ ":mesa_internal_warnings" ]
if (is_clang) {
# Mesa triggers some of these Clang warnings.
@@ -617,7 +626,8 @@ static_library("mesa") {
configs += [ "//build/config/compiler:no_chromium_code" ]
previous_configs = configs
configs = []
- configs = [ ":mesa_internal_config" ] + previous_configs
+ configs = [ ":mesa_internal_config" ] + previous_configs +
+ [ ":mesa_internal_warnings" ]
if (is_clang) {
# Mesa triggers some of these Clang warnings.
@@ -665,7 +675,8 @@ if (!is_android) { # TODO(GYP) enable for Android.
]
previous_configs = configs
configs = []
- configs = [ ":mesa_internal_config" ] + previous_configs
+ configs = [ ":mesa_internal_config" ] + previous_configs +
+ [ ":mesa_internal_warnings" ]
include_dirs = [ "src/src/mesa/drivers" ]
« no previous file with comments | « build/config/compiler/BUILD.gn ('k') | tools/gn/docs/cookbook.md » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698