[FFmpeg-devel] [PATCH] Set -mfpmath=sse for cpus that support it

Reimar Döffinger Reimar.Doeffinger
Thu Aug 27 22:01:28 CEST 2009


On Thu, Aug 27, 2009 at 08:36:08PM +0100, M?ns Rullg?rd wrote:
> Vitor Sessak <vitor1001 at gmail.com> writes:
> 
> > Reimar D?ffinger wrote:
> >> On Thu, Aug 27, 2009 at 04:01:53PM +0200, Vitor Sessak wrote:
> >>> M?ns Rullg?rd wrote:
> >>>> You can't use check_cflags there.  Is there any reason to not attempt
> >>>> using this flag with --cpu values?  I imagine a simple check_cflags
> >>>> line together with the other x86 stuff (ebx checks etc) should be
> >>>> enough.
> >>> I can't just call "check_cflags -mfpmath=sse" for every x86 CPU
> >>> because GCC does not fail with "-march=386 -mfpmath=sse", it gives
> >>> a warning and uses 387 (so check_cflags would always add it). But I
> >>> moved it to the more relevant part of the source (new patch
> >>> attached).
> >> Huh? That reads to me like "check_cflags -mfpmath=sse" would do
> >> exactly
> >> the same thing (well, except for some ugly warnings).
> >
> > Yes, but I thought the ugly warnings are not acceptable.
> 
> Add a flag to kill them.

Or add a check_cflags_nowarn that in addition checks if
grep -i warning
A bit ugly but maybe good enough?



More information about the ffmpeg-devel mailing list