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.
test, ci: Use the documented way to test with ThreadSanitizer #19038
issue hebasto opened this issue on May 21, 2020-
hebasto commented at 2:06 PM on May 21, 2020: member
-
MarcoFalke commented at 2:25 PM on May 21, 2020: member
Might also be useful to fix up the docs
doc/developer-notes.md -
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
- MarcoFalke cross-referenced this on May 21, 2020 from issue ci: tsan with -stdlib=libc++-10 by MarcoFalke
-
hebasto commented at 5:58 AM on May 22, 2020: member
Also the
--disable-walletconfigureoption could be mentioned: - fanquake referenced this in commit 575589a62e on Jun 3, 2020
- sidhujag referenced this in commit 07d81bbfb9 on Jun 3, 2020
- hebasto cross-referenced this on Jul 5, 2020 from issue doc: Mention libc++ for ThreadSanitizer by hebasto
-
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)
- jasonbcox referenced this in commit 93237957b8 on Sep 24, 2020
- fanquake added the label Tests on Aug 11, 2022
-
fanquake commented at 2:59 PM on August 11, 2022: member
What is left to be done here?
- MarcoFalke closed this on Aug 11, 2022
- bitcoin locked this on Aug 11, 2023
Contributors
Labels