Enable incremental linking in release component builds

build/config/compiler/BUILD.gn

Index: build/config/compiler/BUILD.gn
diff --git a/build/config/compiler/BUILD.gn b/build/config/compiler/BUILD.gn
index 7f4578d80b614254fad0d8eb080b9df472e851fa..58d1391db6f5313add347c1e8fdf317a3c7ffc59 100644
--- a/build/config/compiler/BUILD.gn
+++ b/build/config/compiler/BUILD.gn
@@ -1239,7 +1239,10 @@ if (is_win) {
       "/Gw",
     ]
   }
-  common_optimize_on_ldflags = [ "/OPT:ICF" ]  # Redundant COMDAT folding.
+  common_optimize_on_ldflags = []
+  if (!is_component_build) {
+    common_optimize_on_ldflags += [ "/OPT:ICF" ]  # Redundant COMDAT folding.

[Peter Kasting, 2016/09/14 01:15:17]

Does this fix https://bugs.chromium.org/p/chromium/issues/detail?id=621236 ?  Or
is that still broken for non-component builds?

[brucedawson, 2016/09/14 17:49:25]

Yes it does fix that. I am not clear on where incremental was getting enabled
for mksnapshot etc. but this change does avoid the contradiction and the
warnings.

+  }
   if (is_official_build) {
     common_optimize_on_ldflags += [
       "/OPT:REF",  # Remove unreferenced data.