> What type of change is your colleague proposing to turn it on? He was proposing something like this:
if(WIN32) set(CMAKE_CXX_FLAGS "\MP") endif() But neither of us know cmake, he's just trying to make it work the way the original builds did. > Have they read > https://blog.kitware.com/cmake-building-with-all-your-cores/ ? There > are some caveats there which are worth noting (relating to /MP) which > make adding it by default potentially bad since in combination with > target-level parallelism it can greatly exceed the resources of the > machine and break the build. I'll point him at it. If there's a reason, that's fine, we can always turn it on ourselves. > It's also quite possible that xerces is > already being built in parallel with other projects [we do this] so > enabling parallelisation at the lowest level can break the > parallelisation already in place at a much higher level. We don't do that (pretty much can't, the stack is close to vertical for us), so not familiar with the issues. > If there's a way of safely enabling it, that would be great, but it > appears the current recommendations are to configure this when running > cmake rather than hardcoding the behaviour. That's all I needed to know, thanks. We're getting our testing done, so I probably will aim to call for a vote soon, sorry for the hold up. -- Scott --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
