Skip to content

Commit

Permalink
Do not run Valgrind tests in PR jobs
Browse files Browse the repository at this point in the history
Co-authored-by: Dave Rodgman <dave.rodgman@arm.com>
Signed-off-by: Bence Szépkúti <bence.szepkuti@arm.com>
  • Loading branch information
bensze01 and daverodgman committed Dec 6, 2023
1 parent 7655fd2 commit e1c6c4c
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions tests/scripts/all.sh
Original file line number Diff line number Diff line change
Expand Up @@ -1629,7 +1629,7 @@ component_test_memsan_constant_flow () {
make test
}
component_test_valgrind_constant_flow () {
component_release_test_valgrind_constant_flow () {
# This tests both (1) everything that valgrind's memcheck usually checks
# (heap buffer overflows, use of uninitialized memory, use-after-free,
# etc.) and (2) branches or memory access depending on secret values,
Expand Down Expand Up @@ -3371,7 +3371,7 @@ component_test_memsan () {
fi
}
component_test_valgrind () {
component_release_test_valgrind () {
msg "build: Release (clang)"
# default config, in particular without MBEDTLS_USE_PSA_CRYPTO
CC=clang cmake -D CMAKE_BUILD_TYPE:String=Release .
Expand Down Expand Up @@ -3399,7 +3399,7 @@ component_test_valgrind () {
fi
}
component_test_valgrind_psa () {
component_release_test_valgrind_psa () {
msg "build: Release, full (clang)"
# full config, in particular with MBEDTLS_USE_PSA_CRYPTO
scripts/config.py full
Expand Down

0 comments on commit e1c6c4c

Please sign in to comment.