summaryrefslogtreecommitdiffstats
path: root/configure
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2013-03-11 00:50:01 +0100
committerwm4 <wm4@nowhere>2013-03-11 00:50:01 +0100
commit28de5a789129928f44647d31f31cffe7ac43b8ce (patch)
tree1fabef670b533cbc834015de545ed08cf8d0c370 /configure
parentb1be668b9d6ccfc3e3af67eb52487d56c5d1f360 (diff)
downloadmpv-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.
Diffstat (limited to 'configure')
-rwxr-xr-xconfigure6
1 files changed, 3 insertions, 3 deletions
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