diff options
| author | Paolo Bonzini <pbonzini@redhat.com> | 2018-03-06 11:32:44 +0100 |
|---|---|---|
| committer | Paolo Bonzini <pbonzini@redhat.com> | 2018-03-06 14:01:28 +0100 |
| commit | 48e56d503e18bd1e8a75463fd7cc1580bf7e7650 (patch) | |
| tree | 724c88f71aa633d09c78cba9d693134913507c09 | |
| parent | 2b9aef6fcd96ba7ed8c1ee723e391901852d344c (diff) | |
| download | focaccia-qemu-48e56d503e18bd1e8a75463fd7cc1580bf7e7650.tar.gz focaccia-qemu-48e56d503e18bd1e8a75463fd7cc1580bf7e7650.zip | |
Revert "build-sys: compile with -Og or -O1 when --enable-debug"
This reverts commit 906548689e37ab6cca1e93b3f8d9327a4e17e8af. Even with -Og, the debug experience is noticeably worse because gdb shows a lot more "<optimised out>" variables and function arguments. Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| -rwxr-xr-x | configure | 15 |
1 files changed, 2 insertions, 13 deletions
diff --git a/configure b/configure index 7ff00f8e81..ebbda236bb 100755 --- a/configure +++ b/configure @@ -5368,19 +5368,8 @@ if test "$gcov" = "yes" ; then LDFLAGS="-fprofile-arcs -ftest-coverage $LDFLAGS" elif test "$fortify_source" = "yes" ; then CFLAGS="-O2 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 $CFLAGS" -elif test "$debug" = "yes"; then - if compile_prog "-Og" ""; then - CFLAGS="-Og $CFLAGS" - elif compile_prog "-O1" ""; then - CFLAGS="-O1 $CFLAGS" - fi - # Workaround GCC false-positive Wuninitialized bugs with Og or O1: - # https://gcc.gnu.org/bugzilla/show_bug.cgi?id=24639 - if cc_has_warning_flag "-Wno-maybe-uninitialized"; then - CFLAGS="-Wno-maybe-uninitialized $CFLAGS" - fi -else - CFLAGS="-O2 $CFLAGS" +elif test "$debug" = "no"; then + CFLAGS="-O2 $CFLAGS" fi if test "$have_asan" = "yes"; then |