From dfc36f682d1d14f6f95f6b94b396dcd7396a4bd1 Mon Sep 17 00:00:00 2001 From: maximusron Date: Tue, 14 May 2024 13:03:49 +0200 Subject: [PATCH] Update ci to use new suppressions --- .github/workflows/ci.yml | 11 ++++------- etc/clang13-valgrind.supp | 0 2 files changed, 4 insertions(+), 7 deletions(-) create mode 100644 etc/clang13-valgrind.supp diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 2a5b5c2a..e44c7890 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -706,13 +706,10 @@ jobs: if [[ "${{ matrix.os }}" == macos-* ]]; then echo "Skipping Valgrind checks on macOS" else - if [[ "${{ matrix.clang-runtime }}" == "17" || "${{ matrix.clang-runtime }}" == "18" ]]; then - echo "Valgrind reports true for clang-runtime 17 or 18, due to memory leaks with LLVM" - valgrind --error-exitcode=1 --suppressions=../etc/valgrind-cppyy-cling.supp python -m pytest -m "not xfail" -v || true - else - echo "Running valgrind on passing tests" - valgrind --error-exitcode=1 --suppressions=../etc/valgrind-cppyy-cling.supp python -m pytest -m "not xfail" -v - fi + echo "Running valgrind on passing tests" + CLANG_VERSION="${{ matrix.clang-runtime }}" + SUPPRESSION_FILE="../etc/clang${CLANG_VERSION}-valgrind.supp" + valgrind --show-error-list=yes --error-exitcode=1 --track-origins=yes --suppressions="${SUPPRESSION_FILE}" --suppressions=../etc/valgrind-cppyy-cling.supp python -m pytest -m "not xfail" -v fi export RETCODE=+$? echo ::endgroup:: diff --git a/etc/clang13-valgrind.supp b/etc/clang13-valgrind.supp new file mode 100644 index 00000000..e69de29b