From 28de5a789129928f44647d31f31cffe7ac43b8ce Mon Sep 17 00:00:00 2001 From: wm4 Date: Mon, 11 Mar 2013 00:50:01 +0100 Subject: configure: remove -fomit-frame-pointer and -ffast-math from CFLAGS -fomit-frame-pointer is enabled by default with recent gcc and clang compilers if -O2 is used. It also breaks debugging when optimization is disabled, so it makes absolutely no sense to have -fomit-frame-pointer explicitly in the CFLAGS. Get rid of -ffast-math too. It's little more than cargo-culting, and might actually break NaN handling and such things. --- configure | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'configure') diff --git a/configure b/configure index ed981bb21c..d2beb5fce2 100755 --- a/configure +++ b/configure @@ -1118,7 +1118,7 @@ cflag_check -pipe -I. && _pipe="-pipe" && echores "yes" || echores "no" # Checking for CFLAGS if test -z "$CFLAGS" ; then if test "$cc_vendor" = "intel" ; then - CFLAGS="$_opt $_debug $_pipe -fomit-frame-pointer" + CFLAGS="$_opt $_debug $_pipe" WARNFLAGS="-wd167 -wd556 -wd144" elif test "$cc_vendor" = "clang"; then CFLAGS="$_opt $_debug $_pipe" @@ -1127,10 +1127,10 @@ if test -z "$CFLAGS" ; then elif test "$cc_vendor" != "gnu" ; then CFLAGS="$_opt $_debug $_pipe" else - CFLAGS="$_opt $_debug $_pipe -ffast-math -fomit-frame-pointer" + CFLAGS="$_opt $_debug $_pipe" WARNFLAGS="-Wall -Wno-switch -Wno-parentheses -Wpointer-arith -Wredundant-decls" ERRORFLAGS="-Werror-implicit-function-declaration" - extra_ldflags="$extra_ldflags -ffast-math" + extra_ldflags="$extra_ldflags" fi else warn_cflags=yes -- cgit v1.2.3