Skip to content

Commit fb87eb6

Browse files
committed
Enable side-effect free check in CI
1 parent b657f53 commit fb87eb6

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

.cirrus.yml

+6-1
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ env:
1717
BENCH: yes
1818
ITERS: 2
1919
MAKEFLAGS: -j2
20+
CHECK_SIDE_EFFECT_FREE: yes
2021

2122
cat_logs_snippet: &CAT_LOGS
2223
always:
@@ -64,7 +65,7 @@ task:
6465
- env: {BIGNUM: no, STATICPRECOMPUTATION: no}
6566
- env: {BUILD: distcheck, WITH_VALGRIND: no, CTIMETEST: no, BENCH: no}
6667
- env: {CPPFLAGS: -DDETERMINISTIC}
67-
- env: {CFLAGS: -O0, CTIMETEST: no}
68+
- env: {CFLAGS: -O0, CTIMETEST: no, CHECK_SIDE_EFFECT_FREE: no}
6869
- env:
6970
CFLAGS: "-fsanitize=undefined -fno-omit-frame-pointer"
7071
LDFLAGS: "-fsanitize=undefined -fno-omit-frame-pointer"
@@ -76,6 +77,7 @@ task:
7677
EXPERIMENTAL: yes
7778
SCHNORRSIG: yes
7879
CTIMETEST: no
80+
CHECK_SIDE_EFFECT_FREE: auto
7981
- env: { ECMULTGENPRECISION: 2 }
8082
- env: { ECMULTGENPRECISION: 8 }
8183
- env:
@@ -93,6 +95,7 @@ task:
9395
CC: gcc
9496
- env:
9597
CC: clang
98+
CHECK_SIDE_EFFECT_FREE: auto
9699
<< : *MERGE_BASE
97100
test_script:
98101
- ./ci/cirrus.sh
@@ -115,6 +118,7 @@ task:
115118
CC: i686-linux-gnu-gcc
116119
- env:
117120
CC: clang --target=i686-pc-linux-gnu -isystem /usr/i686-linux-gnu/include
121+
CHECK_SIDE_EFFECT_FREE: auto
118122
matrix:
119123
- env:
120124
BIGNUM: gmp
@@ -141,6 +145,7 @@ task:
141145
CC: gcc-9
142146
- env:
143147
CC: clang
148+
CHECK_SIDE_EFFECT_FREE: auto
144149
# Update Command Line Tools
145150
# Uncomment this if the Command Line Tools on the CirrusCI macOS image are too old to brew valgrind.
146151
# See https://apple.stackexchange.com/a/195963 for the implementation.

ci/cirrus.sh

+1
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ valgrind --version || true
1919
--enable-module-ecdh="$ECDH" --enable-module-recovery="$RECOVERY" \
2020
--enable-module-schnorrsig="$SCHNORRSIG" \
2121
--with-valgrind="$WITH_VALGRIND" \
22+
--enable-side-effect-free-check="$CHECK_SIDE_EFFECT_FREE" \
2223
--host="$HOST" $EXTRAFLAGS
2324

2425
# We have set "-j<n>" in MAKEFLAGS.

0 commit comments

Comments
 (0)