<div dir="ltr"><div>I think we should add it, I think it would benefit a lot of users.</div><div dir="ltr"><br></div><div dir="ltr">On Thu, Apr 25, 2019 at 7:34 AM Alexandre Ganea <<a href="mailto:alexandre.ganea@ubisoft.com">alexandre.ganea@ubisoft.com</a>> wrote:</div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div lang="FR-CA"><div class="gmail-m_6437418947644486654WordSection1">
<p class="MsoNormal"><span lang="EN-CA">Most of the changes in the demo patch in [2] are: modernization of the process launching API (ProcessInfo), and support for sys::WaitMany, which I’d like to commit separately and incrementally.<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-CA">Support for /MP is very small and mostly located in clang/trunk/lib/Driver/Compilation.cpp in the demo patch.</span></p></div></div></blockquote><div><br></div><div>+1 for incremental patches<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div lang="FR-CA"><div class="gmail-m_6437418947644486654WordSection1">
<p class="MsoNormal"><span lang="EN-CA">We could subsequently add a new flag –j to the regular clang driver, which mimics /MP. Further down the road, I’d like to discuss optional support for concrt multithreading which, from some preliminary testing, would
 be much faster than the current cc1 sub-process invocation, at least on Windows 10 1703+.</span></p></div></div></blockquote><div><br></div><div>Some people feel strongly that the compiler should not have a hard dependency on threading libraries, so I'm not sure we should go this far. The -cc1 process separation is mainly used for crash recovery and for compiler developers to separate the filling in of a bunch of default header search paths and flags from the user interface. I think as long as we implement some kind of crash recovery mechanism, I'd be in favor of eliminating the -cc1 process for most compiles.</div></div></div>