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

Unified Diff: build/common.gypi

Issue 160408: Make the Visual Studio /MP flag configurable. (Closed)
Patch Set: Created 11 years, 5 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index bd472e576a084e6c670dd8d175be7084d61a3542..1040c0f02f0822f28970d0b23aa2580d4769f25f 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -72,6 +72,10 @@
# but that doesn't work as we'd like.
'msvs_debug_link_incremental%': '2',
+ # Whether to use multiple cores to compile with visual studio. This is
+ # optional because it sometimes causes corruption on VS 2005.
+ 'msvs_multi_core_compile%': '',
+
# The architecture that we're building on.
'target_arch%': 'ia32',
@@ -520,6 +524,10 @@
'WarningLevel': '3',
'WarnAsError': 'true',
'DebugInformationFormat': '3',
+ 'conditions': [
+ [ 'msvs_multi_core_compile',
+ {'AdditionalOptions': '/MP'}, ],
+ ],
},
'VCLibrarianTool': {
'AdditionalOptions': '/ignore:4221',
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698