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

Unified Diff: testing/gtest.gyp

Issue 1774123005: Support clang-cl build on Windows. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Created 4 years, 9 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/standalone.gypi ('k') | third_party/third_party.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: testing/gtest.gyp
diff --git a/testing/gtest.gyp b/testing/gtest.gyp
index 3c01552c4c49a5478596bbb2266e45f1ce28f733..563f340e996cbf686d21425581af4e255e29ec6f 100644
--- a/testing/gtest.gyp
+++ b/testing/gtest.gyp
@@ -142,6 +142,16 @@
],
'msvs_disabled_warnings': [4800],
},
+ 'variables': {
+ 'clang_warning_flags': [
+ # The Mutex constructor initializer list in gtest-port.cc is
+ # incorrectly ordered. See
+ # https://groups.google.com/d/msg/googletestframework/S5uSV8L2TX8/U1FaTDa6J6sJ.
+ '-Wno-reorder',
+ # Suppress warnings for unused constants.
+ '-Wno-unused'
+ ],
+ },
},
{
'target_name': 'gtest_main',
« no previous file with comments | « build/standalone.gypi ('k') | third_party/third_party.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698