RepoMirrors/mpv
1
0
Fork 0
mirror of https://github.com/mpv-player/mpv synced 2025-01-28 02:23:06 +00:00
Code Issues Releases Wiki Activity

configure: --enable-debug: don't change other compiler flags

Browse Source 
The --enable-debug and --enable-profile options set their own compiler
flags, completely different from normal flag selection. These flags
sucked; especially '-W' (an obsolete alias for '-Wextra') generated a
huge number of irrelevant warnings. Change configure to only add "-g"
or similar to the flags that would be used otherwise.
This commit is contained in:
Uoti Urpala 2012-04-19 17:05:50 +03:00
parent deffd15a05
commit e48c6f446c
1 changed files with 8 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
configure vendored
View File

@ -2310,23 +2310,22 @@ fi # if darwin && test "$cc_vendor" = "gnu" ; then
# Checking for CFLAGS # Checking for CFLAGS
_install_strip="-s" _install_strip="-s"
if test "$_profile" != "" || test "$_debug" != "" ; then if test "$_profile" != "" || test "$_debug" != "" ; then
CFLAGS="-O2 $_march $_mcpu $_pipe $_debug $_profile"
WARNFLAGS="-W -Wall"
_install_strip= _install_strip=
elif test -z "$CFLAGS" ; then fi
if test -z "$CFLAGS" ; then
if test "$cc_vendor" = "intel" ; then if test "$cc_vendor" = "intel" ; then
CFLAGS="-O2 $_march $_mcpu $_pipe -fomit-frame-pointer" CFLAGS="-O2 $_debug $_profile $_march $_mcpu $_pipe -fomit-frame-pointer"
WARNFLAGS="-wd167 -wd556 -wd144" WARNFLAGS="-wd167 -wd556 -wd144"
elif test "$cc_vendor" = "sun" ; then elif test "$cc_vendor" = "sun" ; then
CFLAGS="-O2 $_march $_mcpu $_pipe -xc99 -xregs=frameptr" CFLAGS="-O2 $_debug $_profile $_march $_mcpu $_pipe -xc99 -xregs=frameptr"
elif test "$cc_vendor" = "clang"; then elif test "$cc_vendor" = "clang"; then
CFLAGS="-O2 $_march $_pipe" CFLAGS="-O2 $_debug $_profile $_march $_pipe"
WARNFLAGS="-Wall -Wno-switch-enum -Wno-logical-op-parentheses -Wpointer-arith -Wundef -Wno-pointer-sign -Wmissing-prototypes" WARNFLAGS="-Wall -Wno-switch-enum -Wno-logical-op-parentheses -Wpointer-arith -Wundef -Wno-pointer-sign -Wmissing-prototypes"
ERRORFLAGS="-Werror=implicit-function-declaration" ERRORFLAGS="-Werror=implicit-function-declaration"
elif test "$cc_vendor" != "gnu" ; then elif test "$cc_vendor" != "gnu" ; then
CFLAGS="-O2 $_march $_mcpu $_pipe" CFLAGS="-O2 $_debug $_profile $_march $_mcpu $_pipe"
else else
CFLAGS="-O2 $_march $_mcpu $_pipe -ffast-math -fomit-frame-pointer" CFLAGS="-O2 $_debug $_profile $_march $_mcpu $_pipe -ffast-math -fomit-frame-pointer"
WARNFLAGS="-Wall -Wno-switch -Wno-parentheses -Wpointer-arith -Wredundant-decls" WARNFLAGS="-Wall -Wno-switch -Wno-parentheses -Wpointer-arith -Wredundant-decls"
ERRORFLAGS="-Werror-implicit-function-declaration" ERRORFLAGS="-Werror-implicit-function-declaration"
extra_ldflags="$extra_ldflags -ffast-math" extra_ldflags="$extra_ldflags -ffast-math"
@ -6844,12 +6843,8 @@ $def_yasm
#define HAVE_INLINE_ASM 1 #define HAVE_INLINE_ASM 1
/* Use these registers in FFmpeg x86 inline asm. No proper detection yet. */ /* Use these registers in x86 inline asm. No proper detection yet. */
#ifndef MP_DEBUG
#define HAVE_EBP_AVAILABLE 1 #define HAVE_EBP_AVAILABLE 1
#else
#define HAVE_EBP_AVAILABLE 0
#endif
#endif /* MPLAYER_CONFIG_H */ #endif /* MPLAYER_CONFIG_H */
EOF EOF