Skip to content

Commit 938d9c3

Browse files
committed
[valgrind] Remove the expensive checks.
1 parent e46a055 commit 938d9c3

1 file changed

Lines changed: 1 addition & 9 deletions

File tree

test/CMakeLists.txt

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -90,16 +90,8 @@ option(CLAD_CUDA_TEST_USE_SANITIZER "Run Clang cuda tests with compute-sanitizer
9090
option(CLAD_TEST_USE_VG "Run Clang tests under Valgrind" OFF)
9191
set(CLAD_TEST_EXTRA_ARGS --verbose --show-skipped --show-unsupported)
9292
if(CLAD_TEST_USE_VG)
93-
# Run under memcheck. The only errors an LLVM 22 x86_64 build produces on
94-
# this suite are Memcheck's false "Conditional jump depends on uninitialised
95-
# value" for clang's unsigned-compare lowering of a partially-defined
96-
# bitfield word (llvm/llvm-project#194147); every one tops out in one of two
97-
# clang functions, none in a clad frame. clad-valgrind-llvm22.supp silences
98-
# exactly those two frames and nothing else, so a genuine uninitialised-value
99-
# error anywhere still fails the row. Drop the file once the running Valgrind
100-
# carries the CmpLTU/CmpLEU expensive-definedness fix.
10193
set(CLAD_TEST_EXTRA_ARGS ${CLAD_TEST_EXTRA_ARGS}
102-
"--vg --vg-arg=-q --vg-arg=--suppressions=${CMAKE_CURRENT_SOURCE_DIR}/clad-valgrind-llvm22.supp --vg-arg=--expensive-definedness-checks=yes")
94+
"--vg --vg-arg=-q --vg-arg=--suppressions=${CMAKE_CURRENT_SOURCE_DIR}/clad-valgrind-llvm22.supp")
10395
endif ()
10496
if(CLAD_CUDA_TEST_USE_SANITIZER)
10597
set(CLAD_TEST_EXTRA_ARGS ${CLAD_TEST_EXTRA_ARGS} "--param" "cuda_sanitizer=1")

0 commit comments

Comments
 (0)