[PATCH] D76240: [Sanitizers][Test] If the default linker is LLD, disable threads

[Nemanja Ivanovic via Phabricator via llvm-commits]

nemanjai added a comment.

In D76240#1925329 <https://reviews.llvm.org/D76240#1925329>, @MaskRay wrote:

> The -Wl,--no-threads logic seems test machine specific. Shouldn't a site-wise configuration file added somewhere on the test machine to adapt the behavior of lit substitutions? I don't know how to do that, though. lit.site.cfg.py files are generated in the build directory. They are not used for customization.


I don't really understand this comment. In what way is this machine-specific? If each instance of `lld` will spawn `n` threads and lit will fork off as many processes as there are CPU's reported by the system - call this number `m`, then there will be up to `n * m` threads spawned at the same time. On my machine (which I don't imagine is unique), `m == 192` and if I am interpreting the comment in `Parallel.h` correctly, `n == 1024` which makes the upper limit on any similar machine `196608` threads.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76240