[PATCH] D76240: [Sanitizers][Test] If the default linker is LLD, disable threads
Nemanja Ivanovic via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 19 09:11:26 PDT 2020
nemanjai added a comment.
In D76240#1930775 <https://reviews.llvm.org/D76240#1930775>, @grimar wrote:
> 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`.
>
>
> BTW, can't testing with `--no-threads` hide possible LLDs concurency bugs?
> We probably do not want to test LLD with `--(no-)threads` in most cases because it is not a real world use case of LLD.
I agree that this reduces the overall use of LLD with threading enabled while testing. However, I wouldn't characterize this as reducing test coverage for LLD because:
- This does not affect the actual LLD tests
- I would imagine that having a sanitizer test fail intermittently due to a bug caused by LLD's concurrency would actually be unlikely to lead the developer into an investigation of LLD, but rather to a waste of time investigating the sanitizer
I assume there aren't a large number of developers that build with `-DCLANG_DEFAULT_LINKER=lld`. For those that do, I think there are plenty of opportunities to test LLD through using the built compiler so that the loss of testing within the highly parallel `ninja check-all` testing of sanitizers is not that significant.
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