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 1880113003: Enable /GL for all targets in Official (behind a flag) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase - Add a flag to enable this. Created 4 years, 8 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
Index: third_party/mesa/BUILD.gn
diff --git a/third_party/mesa/BUILD.gn b/third_party/mesa/BUILD.gn
index 01e4ad427236040984b150e870cab850d00fe462..0c90820315b343905849a6df0f9558d244764fc3 100644
--- a/third_party/mesa/BUILD.gn
+++ b/third_party/mesa/BUILD.gn
@@ -144,6 +144,11 @@ config("mesa_internal_warnings") {
"/wd4345", # POD-type default initializers.
"/wd4311", # Pointer truncation TODO(brucedawson): http://crbug.com/554200
]
+ if (is_official_build && full_wpo_on_official) {
+ # TODO(sebmarchand): http://crbug.com/604838 This should be removed if
+ # Mesa is ever rolled and the warnings is fixed.
brucedawson 2016/04/29 00:06:13 nit: warnings -> warning (or is -> are)
Sébastien Marchand 2016/04/29 14:15:39 Done.
+ cflags += [ "/wd4723" ] # potential divide by 0.
+ }
}
}

Powered by Google App Engine
This is Rietveld 408576698