[PATCH] [lld][Driver] Add maximum concurrency option.

Michael Spencer bigcheesegs at gmail.com
Sat Apr 20 15:00:29 PDT 2013



================
Comment at: include/lld/Core/Parallel.h:87
@@ +86,3 @@
+
+inline unsigned getDefaultExecutorMaxConcurrency() {
+  return detail::getDefaultExecutorMaxConcurrency();
----------------
Shankar Kalpathi Easwaran wrote:
> this could be a const function.
Non-member function's can't be const.

================
Comment at: lib/Driver/GnuLdDriver.cpp:200
@@ +199,3 @@
+    StringRef(Arg->getValue()).getAsInteger(10, maxConcurrency);
+    options->setMaxConcurrency(maxConcurrency);
+  }
----------------
Shankar Kalpathi Easwaran wrote:
> There might be a user error if the concurrency he is trying to set is greater than the maximum concurrency allowed. Do you think this should be checked as well ?
This is where the max concurrency is set. There is no max system logical concurrency other than the maximum number of threads a process can have.


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



More information about the llvm-commits mailing list