mirror/mpv
0
0
Fork 0
mirror of https://github.com/mpv-player/mpv.git synced 2024-09-20 12:02:23 +02:00
Code

configure: remove -fomit-frame-pointer and -ffast-math from CFLAGS

Browse Source 
-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.
This commit is contained in:
wm4 2013-03-11 00:50:01 +01:00
parent b1be668b9d
commit 28de5a7891
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
configure vendored
View File

@ -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