diff options
| author | Tom Tromey <tom@tromey.com> | 2018-09-16 12:38:12 -0600 |
|---|---|---|
| committer | Tom Tromey <tom@tromey.com> | 2018-09-17 13:51:38 -0600 |
| commit | ae292b3afc0fce65c12014d2cc4621d7a0e661fd (patch) | |
| tree | 700c8b4115795574d20a5f5cf5caef482dba11f3 /gdb | |
| parent | da658607ed24bc5a4dcfcfb56b6e1440ccab33d2 (diff) | |
| download | fsf-binutils-gdb-ae292b3afc0fce65c12014d2cc4621d7a0e661fd.zip fsf-binutils-gdb-ae292b3afc0fce65c12014d2cc4621d7a0e661fd.tar.gz fsf-binutils-gdb-ae292b3afc0fce65c12014d2cc4621d7a0e661fd.tar.bz2 | |
Do not pass -DNDEBUG to Python compilations in development mode
The Python CFLAGS include -DNDEBUG. This was apparently done
intentionally -- setting the flags is done manually because, according
to a comment, python-config passes too many things to the compiler
(which is true).
Per PR python/20445, this patch changes configure so that -DNDEBUG is
only used by release builds. This probably doesn't have very much
effect in practice, but I did see that some Python headers use assert,
so perhaps it will give some safety.
Tested by rebuilding and re-running gdb.python/*.exp on x86-64 Fedora 28.
gdb/ChangeLog
2018-09-17 Tom Tromey <tom@tromey.com>
PR python/20445:
* configure: Rebuild.
* configure.ac: Conditionally use -DNDEBUG for Python.
Diffstat (limited to 'gdb')
| -rw-r--r-- | gdb/ChangeLog | 6 | ||||
| -rwxr-xr-x | gdb/configure | 6 | ||||
| -rw-r--r-- | gdb/configure.ac | 6 |
3 files changed, 16 insertions, 2 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 43c4753..e984f53 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,11 @@ 2018-09-17 Tom Tromey <tom@tromey.com> + PR python/20445: + * configure: Rebuild. + * configure.ac: Conditionally use -DNDEBUG for Python. + +2018-09-17 Tom Tromey <tom@tromey.com> + * configure: Rebuild. * configure.ac: Use gmp as a library dependency when checking for mpfr. diff --git a/gdb/configure b/gdb/configure index e011b77..a5d6e74 100755 --- a/gdb/configure +++ b/gdb/configure @@ -10762,7 +10762,11 @@ $as_echo "#define HAVE_PYTHON 1" >>confdefs.h # would make the python-related objects be compiled differently from the # rest of GDB (e.g., -O2 and -fPIC). if test "${GCC}" = yes; then - tentative_python_cflags="-fno-strict-aliasing -DNDEBUG -fwrapv" + tentative_python_cflags="-fno-strict-aliasing -fwrapv" + # Python headers recommend -DNDEBUG, but it's unclear if that just + # refers to building Python itself. In release mode, though, it + # doesn't hurt for the Python code in gdb to follow. + $development || tentative_python_cflags="$tentative_python_cflags -DNDEBUG" fi if test "x${tentative_python_cflags}" != x; then diff --git a/gdb/configure.ac b/gdb/configure.ac index f658da8..66fc6c6 100644 --- a/gdb/configure.ac +++ b/gdb/configure.ac @@ -965,7 +965,11 @@ if test "${have_libpython}" != no; then # would make the python-related objects be compiled differently from the # rest of GDB (e.g., -O2 and -fPIC). if test "${GCC}" = yes; then - tentative_python_cflags="-fno-strict-aliasing -DNDEBUG -fwrapv" + tentative_python_cflags="-fno-strict-aliasing -fwrapv" + # Python headers recommend -DNDEBUG, but it's unclear if that just + # refers to building Python itself. In release mode, though, it + # doesn't hurt for the Python code in gdb to follow. + $development || tentative_python_cflags="$tentative_python_cflags -DNDEBUG" fi if test "x${tentative_python_cflags}" != x; then |