Re: [FFmpeg-devel] [PATCH 2/2] Don't use CMOV in MMX code

2014-09-15 Thread Calvin Walton
On Fri, 2014-09-12 at 14:53 -0700, Daniel Kang wrote: > > > > >All processors supporing mmx2 also support cmov, so if a test for > > mmx2 > succeeds, we can use cmov. > > Since I originally thought mmx => cmov (and apparently am wrong) is > there > official documentation supporting this? > As

Re: [FFmpeg-devel] [PATCH 2/2] Don't use CMOV in MMX code

2014-09-13 Thread Mikulas Patocka
On Fri, 12 Sep 2014, Daniel Kang wrote: > > All processors supporing mmx2 also support cmov, so if a test for mmx2 > > succeeds, we can use cmov. > Since I originally thought mmx => cmov (and apparently am wrong) is there > official documentation supporting this? There is no official document

Re: [FFmpeg-devel] [PATCH 2/2] Don't use CMOV in MMX code

2014-09-12 Thread Hendrik Leppkes
On Fri, Sep 12, 2014 at 11:53 PM, Daniel Kang wrote: >>All processors supporing mmx2 also support cmov, so if a test for mmx2 > succeeds, we can use cmov. > > Since I originally thought mmx => cmov (and apparently am wrong) is there > official documentation supporting this? CMOV was introduced in

Re: [FFmpeg-devel] [PATCH 2/2] Don't use CMOV in MMX code

2014-09-12 Thread Daniel Kang
>All processors supporing mmx2 also support cmov, so if a test for mmx2 succeeds, we can use cmov. Since I originally thought mmx => cmov (and apparently am wrong) is there official documentation supporting this? ___ ffmpeg-devel mailing list ffmpeg-deve

Re: [FFmpeg-devel] [PATCH 2/2] Don't use CMOV in MMX code

2014-09-12 Thread James Almer
On 12/09/14 4:50 PM, Mikulas Patocka wrote: > There are MMX processors that don't support CMOV (pentium-mmx, amd-k6, cyrix > 6x86). Therefore, we must not use cmov in in MMX code. > > All processors supporing mmx2 also support cmov, so if a test for mmx2 > succeeds, > we can use cmov. > > --- >

[FFmpeg-devel] [PATCH 2/2] Don't use CMOV in MMX code

2014-09-12 Thread Mikulas Patocka
There are MMX processors that don't support CMOV (pentium-mmx, amd-k6, cyrix 6x86). Therefore, we must not use cmov in in MMX code. All processors supporing mmx2 also support cmov, so if a test for mmx2 succeeds, we can use cmov. --- libavcodec/x86/h264_idct.asm |6 ++ 1 file changed, 6