[compiler-rt] 2a73b7b - [NFC][LSAN] Limit the number of concurrent threads is the test

[Florian Hahn via llvm-commits]

Hi,


> On May 11, 2021, at 20:33, Vitaly Buka via llvm-commits <llvm-commits at lists.llvm.org> wrote:
> 
> 
> Author: Vitaly Buka
> Date: 2021-05-11T12:32:53-07:00
> New Revision: 2a73b7bd8cf7620fc0e478ac838b07ee6649dd8a
> 
> URL: https://github.com/llvm/llvm-project/commit/2a73b7bd8cf7620fc0e478ac838b07ee6649dd8a
> DIFF: https://github.com/llvm/llvm-project/commit/2a73b7bd8cf7620fc0e478ac838b07ee6649dd8a.diff
> 
> LOG: [NFC][LSAN] Limit the number of concurrent threads is the test
> 
> Test still fails with D88184 reverted.
> 
> The test was flaky on https://bugs.chromium.org/p/chromium/issues/detail?id=1206745 and
> https://lab.llvm.org/buildbot/#/builders/sanitizer-x86_64-linux
> 
> Reviewed By: morehouse
> 


I think this patch is very likely to cause the tests below to fail on GreenDragon:

      LeakSanitizer-AddressSanitizer-x86_64 :: TestCases/many_threads_detach.cpp
      LeakSanitizer-Standalone-x86_64 :: TestCases/many_threads_detach.cpp

First failure:
      http://green.lab.llvm.org/green/job/clang-stage1-RA/20754/

Still failing in latest build:
      http://green.lab.llvm.org/green/job/clang-stage1-RA/20928/


I reverted the change for now (commit below)

commit 65936b952964d257c387ce2a7a9362a2ce297a63 
Author: Florian Hahn <flo at fhahn.com>
Date:   Mon May 17 09:29:49 2021 +0100

    Revert "[NFC][LSAN] Limit the number of concurrent threads is the test”



Cheers,
Florian