Skip to content

Commit 4e365b0

Browse files
committed
ci: Use -Wthread-safety not -Wthread-safety-analysis
Use -Wthread-safety not -Wthread-safety-analysis in llvm and sanitize jobs. -Wthread-safety is a more general flag which adds additional checks outside the core thread safety analysis. Credit to hodlinator for noticing the bitcoin core build being stricter about thread safety checks than the libmultiprocess build here: #201 (comment)
1 parent 1b8d4a6 commit 4e365b0

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

ci/configs/llvm.bash

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ CI_DESC="CI job using LLVM-based libraries and tools (clang, libc++, clang-tidy,
22
CI_DIR=build-llvm
33
NIX_ARGS=(--arg enableLibcxx true)
44
export CXX=clang++
5-
export CXXFLAGS="-Werror -Wall -Wextra -Wpedantic -Wthread-safety-analysis -Wno-unused-parameter"
5+
export CXXFLAGS="-Werror -Wall -Wextra -Wpedantic -Wthread-safety -Wno-unused-parameter"
66
CMAKE_ARGS=(
77
-G Ninja
88
-DMP_ENABLE_CLANG_TIDY=ON

ci/configs/sanitize.bash

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
CI_DESC="CI job running ThreadSanitizer"
22
CI_DIR=build-sanitize
33
export CXX=clang++
4-
export CXXFLAGS="-ggdb -Werror -Wall -Wextra -Wpedantic -Wthread-safety-analysis -Wno-unused-parameter -fsanitize=thread"
4+
export CXXFLAGS="-ggdb -Werror -Wall -Wextra -Wpedantic -Wthread-safety -Wno-unused-parameter -fsanitize=thread"
55
CMAKE_ARGS=()
66
BUILD_ARGS=(-k -j4)
77
BUILD_TARGETS=(mptest)

0 commit comments

Comments
 (0)