[PATCH] D69825: [Clang][Driver] Re-use the calling process instead of creating a new process for the cc1 invocation

Alexandre Ganea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 26 09:35:24 PST 2019


aganea added a comment.

In D69825#1760373 <https://reviews.llvm.org/D69825#1760373>, @russell.gallop wrote:

> It looks like the git apply didn't work, but the script continued so this was a duff experiment, please ignore. I'll try to fix this and run it again.


Thanks Russell! Can you try running the script overnight? It'll give a better result and we'll see the error bar.

In D69825#1760279 <https://reviews.llvm.org/D69825#1760279>, @russell.gallop wrote:

> Maybe I'm just lucky and none of the process creation problems are affecting me on this system (at this moment).


I'm running tests on 6 differents Windows systems (both desktops and servers), with different OS revisions, and they all behave differently. With this patch, some see better improvements than others. I'll post the results once I have a better picture.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69825/new/

https://reviews.llvm.org/D69825





More information about the llvm-commits mailing list