Skip to content

Commit

Permalink
build: remove /MP from default additonal options
Browse files Browse the repository at this point in the history
PR-URL: nodejs#22661
Reviewed-By: João Reis <[email protected]>
Reviewed-By: Refael Ackermann <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Bartosz Sosnowski <[email protected]>
  • Loading branch information
skelliam authored and tniessen committed Sep 9, 2018
1 parent 82787fb commit 56d9cd4
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions common.gypi
Original file line number Diff line number Diff line change
Expand Up @@ -148,9 +148,9 @@
'MinimalRebuild': 'false',
'OmitFramePointers': 'false',
'BasicRuntimeChecks': 3, # /RTC1
'MultiProcessorCompilation': 'true',
'AdditionalOptions': [
'/bigobj', # prevent error C1128 in VS2015
'/MP', # compile across multiple CPUs
],
},
'VCLinkerTool': {
Expand Down Expand Up @@ -255,8 +255,8 @@
'EnableFunctionLevelLinking': 'true',
'EnableIntrinsicFunctions': 'true',
'RuntimeTypeInfo': 'false',
'MultiProcessorCompilation': 'true',
'AdditionalOptions': [
'/MP', # compile across multiple CPUs
],
}
}
Expand Down

0 comments on commit 56d9cd4

Please sign in to comment.