Skip to content

Commit

Permalink
ci: Run valgrind/memcheck tasks with 2 CPUs
Browse files Browse the repository at this point in the history
... and increase the memory only for UBSan, ASan, LSan builds. Those are
the ones who need more memory.
  • Loading branch information
real-or-random committed Dec 22, 2021
1 parent e70acab commit b4ac1a1
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions .cirrus.yml
Original file line number Diff line number Diff line change
Expand Up @@ -263,8 +263,6 @@ task:
# Sanitizers
task:
<< : *LINUX_CONTAINER
container:
memory: 2G
env:
ECDH: yes
RECOVERY: yes
Expand All @@ -273,11 +271,15 @@ task:
CTIMETEST: no
matrix:
- name: "Valgrind (memcheck)"
container:
cpu: 2
env:
# The `--error-exitcode` is required to make the test fail if valgrind found errors, otherwise it'll return 0 (https://www.valgrind.org/docs/manual/manual-core.html)
WRAPPER_CMD: "valgrind --error-exitcode=42"
SECP256K1_TEST_ITERS: 2
- name: "UBSan, ASan, LSan"
container:
memory: 2G
env:
CFLAGS: "-fsanitize=undefined,address -g"
UBSAN_OPTIONS: "print_stacktrace=1:halt_on_error=1"
Expand Down

0 comments on commit b4ac1a1

Please sign in to comment.