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 21, 2021
1 parent 882f847 commit 965f948
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 965f948

Please sign in to comment.