On Mon, Jun 24, 2013 at 12:22:30PM +0200, Diego Biurrun wrote:
> 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.
Scratch that, patch dropped.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
