win: Stop passing /wd4206 in PCH builds.
We already don't pass it for blink's PCHs and it seems to work fine there.
No intended behavior change, please revert if this breaks you.
BUG=none
Review-Url: https://codereview.chromium.org/2830733006
Cr-Original-Commit-Position: refs/heads/master@{#467353}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: d7119231dcd23626b491d050706871c2729601a3
diff --git a/BUILD.gn b/BUILD.gn
index 593a88d..4a34178 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -359,20 +359,6 @@
# Force include the header.
cflags = [ "/FI$precompiled_header" ]
-
- # Disable warning for "this file was empty after preprocessing". This
- # error is generated only in C mode for ANSI compatibility. It conflicts
- # with precompiled headers since the source file that's "compiled" for
- # making the precompiled header is empty.
- #
- # This error doesn't happen every time. In VS2013, it seems if the .pch
- # file doesn't exist, no error will be generated (probably MS tested this
- # case but forgot the other one?). To reproduce this error, do a build,
- # then delete the precompile.c.obj file, then build again.
- #
- # TODO(sof): determine VS2015 status and retire the setting from all
- # precompiled configurations.
- cflags_c = [ "/wd4206" ]
} else if (is_mac) {
precompiled_source = "//build/precompile.h"
}
