Skip to content

Commit

Permalink
Add --with-assertions configure flag to enable C assertions(python#1731)
Browse files Browse the repository at this point in the history
Defaults to 'no', but as before assertions are implied by --with-pydebug.
  • Loading branch information
Yhg1s authored May 22, 2017
1 parent 0c4aca5 commit ddbfa2c
Show file tree
Hide file tree
Showing 3 changed files with 56 additions and 2 deletions.
4 changes: 4 additions & 0 deletions Misc/NEWS
Original file line number Diff line number Diff line change
Expand Up @@ -1100,6 +1100,10 @@ Documentation
Build
-----

- bpo-29941: Add ``--with-assertions`` configure flag to explicitly enable
C ``assert()`` checks. Defaults to off. ``--with-pydebug`` implies
``--with-assertions``.

- bpo-28787: Fix out-of-tree builds of Python when configured with
``--with--dtrace``.

Expand Down
31 changes: 30 additions & 1 deletion configure
Original file line number Diff line number Diff line change
Expand Up @@ -817,6 +817,7 @@ with_suffix
enable_shared
enable_profiling
with_pydebug
with_assertions
enable_optimizations
with_lto
with_hash_algorithm
Expand Down Expand Up @@ -1512,6 +1513,7 @@ Optional Packages:
compiler
--with-suffix=.exe set executable suffix
--with-pydebug build with Py_DEBUG defined
--with-assertions build with C assertions enabled
--with-lto Enable Link Time Optimization in PGO builds.
Disabled by default.
--with-hash-algorithm=[fnv|siphash24]
Expand Down Expand Up @@ -6464,6 +6466,33 @@ $as_echo "no" >&6; }
fi


# Check for --with-assertions. Py_DEBUG implies assertions, but also changes
# the ABI. This allows enabling assertions without changing the ABI.
assertions='false'
{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for --with-assertions" >&5
$as_echo_n "checking for --with-assertions... " >&6; }

# Check whether --with-assertions was given.
if test "${with_assertions+set}" = set; then :
withval=$with_assertions;
if test "$withval" != no
then
assertions='true'
fi
fi

if test "$assertions" = 'true'; then
{ $as_echo "$as_me:${as_lineno-$LINENO}: result: yes" >&5
$as_echo "yes" >&6; }
elif test "$Py_DEBUG" = 'true'; then
assertions='true'
{ $as_echo "$as_me:${as_lineno-$LINENO}: result: implied by --with-pydebug" >&5
$as_echo "implied by --with-pydebug" >&6; }
else
{ $as_echo "$as_me:${as_lineno-$LINENO}: result: no" >&5
$as_echo "no" >&6; }
fi

# Enable optimization flags


Expand Down Expand Up @@ -7414,7 +7443,7 @@ case "$CC" in
;;
esac

if test "$Py_DEBUG" = 'true'; then
if test "$assertions" = 'true'; then
:
else
OPT="-DNDEBUG $OPT"
Expand Down
23 changes: 22 additions & 1 deletion configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -1251,6 +1251,27 @@ else AC_MSG_RESULT(no); Py_DEBUG='false'
fi],
[AC_MSG_RESULT(no)])

# Check for --with-assertions. Py_DEBUG implies assertions, but also changes
# the ABI. This allows enabling assertions without changing the ABI.
assertions='false'
AC_MSG_CHECKING(for --with-assertions)
AC_ARG_WITH(assertions,
AC_HELP_STRING([--with-assertions], [build with C assertions enabled]),
[
if test "$withval" != no
then
assertions='true'
fi],
[])
if test "$assertions" = 'true'; then
AC_MSG_RESULT(yes)
elif test "$Py_DEBUG" = 'true'; then
assertions='true'
AC_MSG_RESULT(implied by --with-pydebug)
else
AC_MSG_RESULT(no)
fi

# Enable optimization flags
AC_SUBST(DEF_MAKE_ALL_RULE)
AC_SUBST(DEF_MAKE_RULE)
Expand Down Expand Up @@ -1868,7 +1889,7 @@ case "$CC" in
;;
esac

if test "$Py_DEBUG" = 'true'; then
if test "$assertions" = 'true'; then
:
else
OPT="-DNDEBUG $OPT"
Expand Down

0 comments on commit ddbfa2c

Please sign in to comment.