Index: build/config/nacl/BUILD.gn |
diff --git a/build/config/nacl/BUILD.gn b/build/config/nacl/BUILD.gn |
index 84ffc62c6686d328f185e9cf52ce3a9773283718..217494fb1ba5aeb1cbe1ddd0d16b31f56dc40dbc 100644 |
--- a/build/config/nacl/BUILD.gn |
+++ b/build/config/nacl/BUILD.gn |
@@ -25,13 +25,28 @@ config("nexe_defines") { |
] |
} |
+config("nacl_warnings") { |
+ if (is_win) { |
+ # Some NaCl code uses forward declarations of static const variables, |
+ # with initialized definitions later on. (The alternative would be |
+ # many, many more forward declarations of everything used in that |
+ # const variable's initializer before the definition.) The Windows |
+ # compiler is too stupid to notice that there is an initializer later |
+ # in the file, and warns about the forward declaration. |
+ cflags = [ "/wd4132" ] |
+ } |
+} |
+ |
# The base target that all targets in the NaCl build should depend on. |
# This allows configs to be modified for everything in the NaCl build, even when |
# the NaCl build is composed into the Chrome build. (GN has no functionality to |
-# add flags to everythin in //native_client, having a base target works around |
+# add flags to everything in //native_client, having a base target works around |
# that limitation.) |
source_set("nacl_base") { |
- public_configs = [ ":nacl_defines" ] |
+ public_configs = [ |
+ ":nacl_defines", |
+ ":nacl_warnings", |
+ ] |
if (current_os == "nacl") { |
public_configs += [ ":nexe_defines" ] |
} |