[PATCH] D147493: [ELF] Cap parallel::strategy to 8 cores when --threads= is unspecified

[Mateusz Guzik via Phabricator via llvm-commits]

mjguzik added a comment.

I have to repeat the note I made on discourse: even for something embarrassingly parallel, there is so much work which is there to do and it very well may be majority of the threads you spawned remain unused.

Absent an informed way to spawn threads, a sanitizing cap is definitely prudent for thinlto.

I don't know what kind of binaries you use the opt with -- would chrome count? ;) I mean, for something really big, how many threads can really end up being used at the same time?

That said, for my use case the above patch already introduces the damage control so I'm happy to bugger off.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147493