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

2014-09-17 Thread Ronald S. Bultje
Hi, On Wed, Sep 17, 2014 at 4:08 AM, Michael Niedermayer 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

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

2014-09-17 Thread Michael Niedermayer
On Sun, Sep 14, 2014 at 01:01:45AM +0200, Mikulas Patocka wrote: > > > Hi > > > > > > Here I'm sending two patches to fix portability for 586-class machines > > > (Pentium, K6, etc.) > > > > > > > > > If the CPU is generic, 386, 486 or pentium, we must not use cmov in inline > > > assembler. > > >

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

2014-09-13 Thread Mikulas Patocka
> > Hi > > > > Here I'm sending two patches to fix portability for 586-class machines > > (Pentium, K6, etc.) > > > > > > If the CPU is generic, 386, 486 or pentium, we must not use cmov in inline > > assembler. > > > > Note that some Linux distributions are compiled for i686, and for them it is >

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

2014-09-12 Thread Michael Niedermayer
Hi On Fri, Sep 12, 2014 at 09:49:30PM +0200, Mikulas Patocka wrote: > Hi > > Here I'm sending two patches to fix portability for 586-class machines > (Pentium, K6, etc.) > > > If the CPU is generic, 386, 486 or pentium, we must not use cmov in inline > assembler. > [...] > > --- > configure

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

2014-09-12 Thread Hendrik Leppkes
On Fri, Sep 12, 2014 at 9:49 PM, Mikulas Patocka wrote: > Hi > > Here I'm sending two patches to fix portability for 586-class machines > (Pentium, K6, etc.) > > > If the CPU is generic, 386, 486 or pentium, we must not use cmov in inline > assembler. > > Note that some Linux distributions are com

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

2014-09-12 Thread Mikulas Patocka
Hi Here I'm sending two patches to fix portability for 586-class machines (Pentium, K6, etc.) If the CPU is generic, 386, 486 or pentium, we must not use cmov in inline assembler. Note that some Linux distributions are compiled for i686, and for them it is possible to use cmov in the assembler