Index: build/common.gypi |
diff --git a/build/common.gypi b/build/common.gypi |
index f7d82a1f01e21ddee004f458e83754ab69c6d145..d5d5a5960c02e9602d9f23ee3c5ab6f2aa6fe43c 100644 |
--- a/build/common.gypi |
+++ b/build/common.gypi |
@@ -1903,11 +1903,11 @@ |
['asan==1', { |
'win_use_allocator_shim%': 0, |
}], |
- ['buildtype!="Official"', { |
- # Not enabled in Official because it adds padding to builds. |
- # Enabled for both shared_library and static_library on an |
- # experimental basis to see if it improves windows cycle times, |
- # see http://crbug.com/404809. |
+ ['component=="shared_library" and "<(GENERATOR)"=="ninja"', { |
+ # Only enabled by default for ninja because it's buggy in VS. |
+ # Not enabled for component=static_library because some targets |
+ # are too large and the toolchain fails due to the size of the |
+ # .obj files. |
'incremental_chrome_dll%': 1, |
}], |
# Don't do incremental linking for large modules on 32-bit or when |
@@ -3319,17 +3319,6 @@ |
# information is used by the Syzygy optimization tool when |
# decomposing the release image. |
'Profile': 'true', |
- |
- 'conditions': [ |
- ['incremental_chrome_dll', { |
- # During bring-up of Release incremental, have the linker emit |
- # a one line diagnostic describing why incremental linking |
- # failed, when it does. http://crbug.com/404809. |
- 'AdditionalOptions': [ |
- '/verbose:incr', |
- ], |
- }], |
- ], |
}, |
}, |
'conditions': [ |