Skip to content

Commit 9bc3343

Browse files
committed
Merge bitcoin/bitcoin#32999: ci: Use APT_LLVM_V in msan task
fad040a ci: Use APT_LLVM_V in msan task (MarcoFalke) Pull request description: This skips compilation of clang by using the apt. ACKs for top commit: m3dwards: ACK fad040a willcl-ark: ACK fad040a Tree-SHA512: cc8977a0e97f731b15a2bb9321442d4fc935e310a9cd1993c4ec08ddfd8d7f08a128bbe51ad4d820627bbdcdc748dd58feeec00dee6ee0723e528c546d209f92
2 parents 5878f35 + fad040a commit 9bc3343

File tree

5 files changed

+32
-22
lines changed

5 files changed

+32
-22
lines changed

ci/test/00_setup_env_native_asan.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,8 @@ export GOAL="install"
2626
export BITCOIN_CONFIG="\
2727
-DWITH_USDT=ON -DWITH_ZMQ=ON -DBUILD_GUI=ON \
2828
-DSANITIZERS=address,float-divide-by-zero,integer,undefined \
29-
-DCMAKE_C_COMPILER=clang-${APT_LLVM_V} \
30-
-DCMAKE_CXX_COMPILER=clang++-${APT_LLVM_V} \
29+
-DCMAKE_C_COMPILER=clang \
30+
-DCMAKE_CXX_COMPILER=clang++ \
3131
-DCMAKE_C_FLAGS='-ftrivial-auto-var-init=pattern' \
3232
-DCMAKE_CXX_FLAGS='-ftrivial-auto-var-init=pattern' \
3333
-DAPPEND_CXXFLAGS='-std=c++23' \

ci/test/00_setup_env_native_fuzz.sh

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,8 @@ export CI_CONTAINER_CAP="--cap-add SYS_PTRACE" # If run with (ASan + LSan), the
1919
export BITCOIN_CONFIG="\
2020
-DBUILD_FOR_FUZZING=ON \
2121
-DSANITIZERS=fuzzer,address,undefined,float-divide-by-zero,integer \
22-
-DCMAKE_C_COMPILER=clang-${APT_LLVM_V} \
23-
-DCMAKE_CXX_COMPILER=clang++-${APT_LLVM_V} \
22+
-DCMAKE_C_COMPILER=clang \
23+
-DCMAKE_CXX_COMPILER=clang++ \
2424
-DCMAKE_C_FLAGS='-ftrivial-auto-var-init=pattern' \
2525
-DCMAKE_CXX_FLAGS='-ftrivial-auto-var-init=pattern' \
2626
"
27-
export LLVM_SYMBOLIZER_PATH="/usr/bin/llvm-symbolizer-${APT_LLVM_V}"

ci/test/00_setup_env_native_msan.sh

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,13 +7,14 @@
77
export LC_ALL=C.UTF-8
88

99
export CI_IMAGE_NAME_TAG="mirror.gcr.io/ubuntu:24.04"
10+
export APT_LLVM_V="20"
1011
LIBCXX_DIR="/msan/cxx_build/"
1112
export MSAN_FLAGS="-fsanitize=memory -fsanitize-memory-track-origins=2 -fno-omit-frame-pointer -g -O1 -fno-optimize-sibling-calls"
1213
LIBCXX_FLAGS="-nostdinc++ -nostdlib++ -isystem ${LIBCXX_DIR}include/c++/v1 -L${LIBCXX_DIR}lib -Wl,-rpath,${LIBCXX_DIR}lib -lc++ -lc++abi -lpthread -Wno-unused-command-line-argument"
1314
export MSAN_AND_LIBCXX_FLAGS="${MSAN_FLAGS} ${LIBCXX_FLAGS}"
1415

1516
export CONTAINER_NAME="ci_native_msan"
16-
export PACKAGES="ninja-build"
17+
export PACKAGES="clang-${APT_LLVM_V} llvm-${APT_LLVM_V} llvm-${APT_LLVM_V}-dev libclang-${APT_LLVM_V}-dev libclang-rt-${APT_LLVM_V}-dev ninja-build"
1718
export DEP_OPTS="DEBUG=1 NO_QT=1 CC=clang CXX=clang++ CFLAGS='${MSAN_FLAGS}' CXXFLAGS='${MSAN_AND_LIBCXX_FLAGS}'"
1819
export GOAL="install"
1920
# Setting CMAKE_{C,CXX}_FLAGS_DEBUG flags to an empty string ensures that the flags set in MSAN_FLAGS remain unaltered.

ci/test/00_setup_env_native_tsan.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ export CONTAINER_NAME=ci_native_tsan
1010
export CI_IMAGE_NAME_TAG="mirror.gcr.io/ubuntu:24.04"
1111
export APT_LLVM_V="20"
1212
export PACKAGES="clang-${APT_LLVM_V} llvm-${APT_LLVM_V} libclang-rt-${APT_LLVM_V}-dev libc++abi-${APT_LLVM_V}-dev libc++-${APT_LLVM_V}-dev python3-zmq"
13-
export DEP_OPTS="CC=clang-${APT_LLVM_V} CXX='clang++-${APT_LLVM_V} -stdlib=libc++'"
13+
export DEP_OPTS="CC=clang CXX='clang++ -stdlib=libc++'"
1414
export GOAL="install"
1515
export BITCOIN_CONFIG="-DWITH_ZMQ=ON -DSANITIZERS=thread \
1616
-DAPPEND_CPPFLAGS='-DARENA_DEBUG -DDEBUG_LOCKORDER -DDEBUG_LOCKCONTENTION -D_LIBCPP_REMOVE_TRANSITIVE_INCLUDES'"

ci/test/01_base_install.sh

Lines changed: 25 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -44,27 +44,37 @@ elif [ "$CI_OS_NAME" != "macos" ]; then
4444
${CI_RETRY_EXE} bash -c "apt-get install --no-install-recommends --no-upgrade -y $PACKAGES $CI_BASE_PACKAGES"
4545
fi
4646

47+
if [ -n "${APT_LLVM_V}" ]; then
48+
update-alternatives --install /usr/bin/clang++ clang++ "/usr/bin/clang++-${APT_LLVM_V}" 100
49+
update-alternatives --install /usr/bin/clang clang "/usr/bin/clang-${APT_LLVM_V}" 100
50+
update-alternatives --install /usr/bin/llvm-symbolizer llvm-symbolizer "/usr/bin/llvm-symbolizer-${APT_LLVM_V}" 100
51+
fi
52+
4753
if [ -n "$PIP_PACKAGES" ]; then
4854
# shellcheck disable=SC2086
4955
${CI_RETRY_EXE} pip3 install --user $PIP_PACKAGES
5056
fi
5157

5258
if [[ ${USE_MEMORY_SANITIZER} == "true" ]]; then
53-
${CI_RETRY_EXE} git clone --depth=1 https://github.com/llvm/llvm-project -b "llvmorg-20.1.0" /msan/llvm-project
54-
55-
cmake -G Ninja -B /msan/clang_build/ \
56-
-DLLVM_ENABLE_PROJECTS="clang" \
57-
-DCMAKE_BUILD_TYPE=Release \
58-
-DLLVM_TARGETS_TO_BUILD=Native \
59-
-DLLVM_ENABLE_RUNTIMES="compiler-rt;libcxx;libcxxabi;libunwind" \
60-
-S /msan/llvm-project/llvm
61-
62-
ninja -C /msan/clang_build/ "$MAKEJOBS"
63-
ninja -C /msan/clang_build/ install-runtimes
64-
65-
update-alternatives --install /usr/bin/clang++ clang++ /msan/clang_build/bin/clang++ 100
66-
update-alternatives --install /usr/bin/clang clang /msan/clang_build/bin/clang 100
67-
update-alternatives --install /usr/bin/llvm-symbolizer llvm-symbolizer /msan/clang_build/bin/llvm-symbolizer 100
59+
if [ -n "${APT_LLVM_V}" ]; then
60+
${CI_RETRY_EXE} git clone --depth=1 https://github.com/llvm/llvm-project -b "llvmorg-$( clang --version | sed --silent 's@.*clang version \([0-9.]*\).*@\1@p' )" /msan/llvm-project
61+
else
62+
${CI_RETRY_EXE} git clone --depth=1 https://github.com/llvm/llvm-project -b "llvmorg-20.1.8" /msan/llvm-project
63+
64+
cmake -G Ninja -B /msan/clang_build/ \
65+
-DLLVM_ENABLE_PROJECTS="clang" \
66+
-DCMAKE_BUILD_TYPE=Release \
67+
-DLLVM_TARGETS_TO_BUILD=Native \
68+
-DLLVM_ENABLE_RUNTIMES="compiler-rt;libcxx;libcxxabi;libunwind" \
69+
-S /msan/llvm-project/llvm
70+
71+
ninja -C /msan/clang_build/ "$MAKEJOBS"
72+
ninja -C /msan/clang_build/ install-runtimes
73+
74+
update-alternatives --install /usr/bin/clang++ clang++ /msan/clang_build/bin/clang++ 100
75+
update-alternatives --install /usr/bin/clang clang /msan/clang_build/bin/clang 100
76+
update-alternatives --install /usr/bin/llvm-symbolizer llvm-symbolizer /msan/clang_build/bin/llvm-symbolizer 100
77+
fi
6878

6979
cmake -G Ninja -B /msan/cxx_build/ \
7080
-DLLVM_ENABLE_RUNTIMES="libcxx;libcxxabi;libunwind" \

0 commit comments

Comments
 (0)