test, ci: Use the documented way to test with ThreadSanitizer #19038

1. 
   hebasto commented at 2:06 pm on May 21, 2020: member

   According to the ThreadSanitizer docs:

   C++11 threading is supported with llvm libc++.

   To use the ThreadSanitizer in the documented way we should build from depends with clang++ -stdlib=libc++ (see #18820).

   Related to #19024.

2. 
   MarcoFalke commented at 2:25 pm on May 21, 2020: member

   Might also be useful to fix up the docs doc/developer-notes.md
3. 
   MarcoFalke commented at 3:06 pm on May 21, 2020: member

   So I guess there are two things to note:

   • llvm-10 is required
   • llvm libc++ is required
4. 
   hebasto commented at 5:58 am on May 22, 2020: member

   Also the --disable-wallet configure option could be mentioned:

   • #18823 (comment)
   • #18916 (comment)
5. fanquake referenced this in commit 575589a62e on Jun 3, 2020
6. sidhujag referenced this in commit 07d81bbfb9 on Jun 3, 2020
7. 
   MarcoFalke commented at 2:43 pm on July 5, 2020: member

   So the gnu tsan seems to cover a subset of the tsan in the llvm libc++? While it is probably recommended to use llvm 10 libc++, it shouldn’t be a requirement to just run with tsan. (It is only a requirement if you want to get the most sensitivity out of the tsan)
8. jasonbcox referenced this in commit 93237957b8 on Sep 24, 2020
9. fanquake added the label Tests on Aug 11, 2022
10. 
    fanquake commented at 2:59 pm on August 11, 2022: member

    What is left to be done here?
11. MarcoFalke closed this on Aug 11, 2022

    ---

12. bitcoin locked this on Aug 11, 2023

Contributors
hebasto MarcoFalke fanquake

Labels
Tests