[MPlayer-dev-eng] [PATCH] build: Keep compiler-specific warning flags for debug and profiling builds
Alexander Strasser
eclipse7 at gmx.net
Wed Jul 17 00:45:44 CEST 2013
On 2013-07-16 17:32 +0200, Diego Biurrun wrote:
> On 2013-07-15 00:34, Alexander Strasser wrote:
> >Without warning flags like -Werror-implicit-function-declaration
> >some tests like gethostname2 will compile and therefore succeed.
> >But not without causing strange run-time failures later on e.g.
> >when compiled for x86_64 .
> >
> >Having useful warning flags dropped on debug enabled builds seems
> >also counter intuitive to me.
> >
> >This does not necessarily help with all compilers, but that can
> >be fixed if those compilers have the necessary switches to be
> >more strict.
> >
> >--- a/configure
> >+++ b/configure
> >@@ -2705,11 +2705,7 @@ fi # if darwin && test "$cc_vendor" = "gnu" ; then
> >
> > # Checking for CFLAGS
> > _install_strip="-s"
> >-if test "$_profile" != "" || test "$_debug" != "" ; then
> >- CFLAGS="-O2 $_march $_mcpu $_pipe $_debug $_profile"
> >- WARNFLAGS="-W -Wall"
> >- _install_strip=
> >-elif test -z "$CFLAGS" ; then
> >+if test -z "$CFLAGS" || test "$_profile" != "" || test "$_debug" != ""; then
> > if test "$cc_vendor" = "intel" ; then
> > CFLAGS="-O2 $_march $_mcpu $_pipe -fomit-frame-pointer"
> > WARNFLAGS="-wd167 -wd556 -wd144"
> >@@ -2725,6 +2721,12 @@ elif test -z "$CFLAGS" ; then
> > WARN_CFLAGS="-Werror-implicit-function-declaration"
> > extra_ldflags="$extra_ldflags -ffast-math"
> > fi
> >+
> >+ if test "$_profile" != "" || test "$_debug" != ""; then
> >+ CFLAGS="-O2 $_march $_mcpu $_pipe $_debug $_profile"
> >+ WARNFLAGS="-W -Wall $WARNFLAGS"
> >+ _install_strip=
> >+ fi
> > else
> > warn_cflags=yes
> > fi
>
> This is far from fixing the root cause, which is that compiler flags
> handling in configure is a royal mess, but it doesn't make things
> worse for the more common gcc case. This won't work for some or all
> of suncc, icc, clang, but they look just as broken before.
I agree that flags handling is suboptimal; to say it in a nice way.
> Commit if you feel like it ...
Will commit in a few days. I don't like to keep such a trap around.
Alexander
More information about the MPlayer-dev-eng
mailing list