[PATCH] clang-modernize: Introduce process-based parallelism

Manuel Klimek klimek at google.com
Tue Nov 12 02:35:27 PST 2013


  I'd really like something like this in Tooling, so it'd be available for other tools - since it's basically just forwarding tool calls to subprocesses that should be doable.

  I'm also fine with checking this in for clang-modernize first, though, and work on getting the design for Tooling right on the side without blocking clang-modernize...


================
Comment at: clang-modernize/Core/JobDispatcher.cpp:110
@@ +109,3 @@
+        if (I != ExecutingJobs.end()) {
+          CurrentParallelJobs--;
+          AtLeastOneProcessHasFinished = true;
----------------
Shouldn't that be decremented by the number of finished jobs?


http://llvm-reviews.chandlerc.com/D1950



More information about the cfe-commits mailing list