[PATCH] D115019: ThreadPool: grow the pool only as needed

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added inline comments.


================
Comment at: llvm/include/llvm/Support/ThreadPool.h:71
+  // number of threads).
+  unsigned getThreadCount() const { return MaxThreadCount; }
 
----------------
I'm not fond of keeping the API name as-is with a new semantics.

What about removing it and using two APIs instead:

- getMaxThreadCount()
- getAvailableThreadCount() => return Threads.size()


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115019