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

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 26 13:02:46 PDT 2020


MaskRay added a comment.

> In D76240#1927115 <https://reviews.llvm.org/D76240#1927115>, @MaskRay wrote:
>  For lld, I think --threads={0,1,2,...} is better than the current --(no-)threads.

I have a change to rename lld `--no-threads` to `--threads=N`. It takes more efforts to actually work because I need to patch `llvm::parallel::for_each_n` called by lld/include/lld/Common/Threads.h#L75

Can the lit test run a system lld which may not support --threads= after --no-threads is removed?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76240





More information about the llvm-commits mailing list