On Sun, Jun 23, 2013 at 01:37:48PM +0200, Diego Biurrun wrote:
> On 2013-06-20 13:58, Diego Biurrun wrote:
> >--- a/configure
> >+++ b/configure
> >@@ -3360,8 +3360,6 @@ EOF
> >      exit 1;
> >  fi
> >
> >-disabled optimizations || check_cflags -fomit-frame-pointer
> >-
> >  enable_weak_pic() {
> >      disabled pic && return
> >      enable pic
> >@@ -3883,6 +3881,7 @@ if [ -z "$optflags" ]; then
> >          optflags=$cflags_size
> >      elif enabled optimizations; then
> >          optflags=$cflags_speed
> >+        check_cflags -fomit-frame-pointer
> >      else
> >          optflags=$cflags_noopt
> >      fi
> 
> I'm getting second thoughts about this one.  While it groups the
> optimization flags together, it also moves the frame pointer check
> below all the function and header checks, which could have have
> unwanted side effects.  In the long term, all the compiler flags
> should be determined before all the function, header, and library
> checks.

I'll push this and let FATE have a look at the result.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to