Skip to content

Commit 8e79c7e

Browse files
committed
build: Ensure no optimization when building for coverage analysis
1 parent 427bc3c commit 8e79c7e

File tree

1 file changed

+11
-0
lines changed

1 file changed

+11
-0
lines changed

configure.ac

+11
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,11 @@ AM_INIT_AUTOMAKE([1.11.2 foreign subdir-objects])
2929
# Make the compilation flags quiet unless V=1 is used.
3030
m4_ifdef([AM_SILENT_RULES], [AM_SILENT_RULES([yes])])
3131

32+
if test "${CFLAGS+set}" = "set"; then
33+
CFLAGS_overridden=yes
34+
else
35+
CFLAGS_overridden=no
36+
fi
3237
AC_PROG_CC
3338
AM_PROG_AS
3439
AM_PROG_AR
@@ -241,6 +246,12 @@ fi
241246
if test x"$enable_coverage" = x"yes"; then
242247
SECP_CONFIG_DEFINES="$SECP_CONFIG_DEFINES -DCOVERAGE=1"
243248
SECP_CFLAGS="-O0 --coverage $SECP_CFLAGS"
249+
# If coverage is enabled, and the user has not overridden CFLAGS,
250+
# override Autoconf's value "-g -O2" with "-g". Otherwise we'd end up
251+
# with "-O0 --coverage -g -O2".
252+
if test "$CFLAGS_overridden" = "no"; then
253+
CFLAGS="-g"
254+
fi
244255
LDFLAGS="--coverage $LDFLAGS"
245256
else
246257
# Most likely the CFLAGS already contain -O2 because that is autoconf's default.

0 commit comments

Comments
 (0)