diff options
author | wm4 <wm4@nowhere> | 2013-03-11 00:50:01 +0100 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2013-03-11 00:50:01 +0100 |
commit | 28de5a789129928f44647d31f31cffe7ac43b8ce (patch) | |
tree | 1fabef670b533cbc834015de545ed08cf8d0c370 | |
parent | b1be668b9d6ccfc3e3af67eb52487d56c5d1f360 (diff) | |
download | mpv-28de5a789129928f44647d31f31cffe7ac43b8ce.tar.bz2 mpv-28de5a789129928f44647d31f31cffe7ac43b8ce.tar.xz |
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.
-rwxr-xr-x | configure | 6 |
1 files changed, 3 insertions, 3 deletions
@@ -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 |