[PATCH] D122922: [lld][common][lld-macho] Support per-thread allocators and StringSavers

Thorsten via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 6 12:34:58 PDT 2022


tschuett added a comment.

In D122922#3433979 <https://reviews.llvm.org/D122922#3433979>, @int3 wrote:

>> You cannot query the maximum parallelism ondemand and then create the array?
>
> It's a little fragile -- things would break if someone created more than one ThreadPool instance & gave each instance the max hardware parallelism

But then `MAX_THREADS ` has the same problem? For two thread pools I would need `2 * MAX_THREADS` ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122922



More information about the llvm-commits mailing list