[FFmpeg-devel] [PATCH 1/2] Fix miscompilation for i586.

Ronald S. Bultje rsbultje at gmail.com
Wed Sep 17 13:34:17 CEST 2014


Hi,

On Wed, Sep 17, 2014 at 4:08 AM, Michael Niedermayer <michaelni at gmx.at>
wrote:

> On Sun, Sep 14, 2014 at 01:01:45AM +0200, Mikulas Patocka wrote:
>
> In the program, you can easily determine what -march was passed to the
> > compiler and react to it - if one of __i686__, __athlon__, __SSE__ is
> > defined, you can use cmov, because the compiler is already generating
> > cmov.
>
> thats true,
> but maybe its better if you put this check in configure so the
> --enable/disable flags can be used to override it


+1 on this. For code that is clearly intended to be shared between
compilers (and many compilers support inline asm), I don't think we want to
use compiler-specific flags.

Ronald


More information about the ffmpeg-devel mailing list