Re: [FFmpeg-devel] [PATCH] x86/imdct36: use extractps inside the STORE macro

2016-01-28 Thread James Almer
On 1/28/2016 1:25 PM, Henrik Gramner wrote: > On Mon, Jan 25, 2016 at 7:41 PM, James Almer wrote: >> +%if cpuflag(sse4) >> +extractps [%3 ], %1, 0 >> +extractps [%3 + %4], %1, 1 >> +extractps [%3 + 2*%4], %1, 2 >> +extractps [%3 + 3*%4], %1, 3 > > extractps with imm=0 shou

Re: [FFmpeg-devel] [PATCH] x86/imdct36: use extractps inside the STORE macro

2016-01-28 Thread Henrik Gramner
On Mon, Jan 25, 2016 at 7:41 PM, James Almer wrote: > +%if cpuflag(sse4) > +extractps [%3 ], %1, 0 > +extractps [%3 + %4], %1, 1 > +extractps [%3 + 2*%4], %1, 2 > +extractps [%3 + 3*%4], %1, 3 extractps with imm=0 should be replaced with movss.

Re: [FFmpeg-devel] [PATCH] x86/imdct36: use extractps inside the STORE macro

2016-01-28 Thread Michael Niedermayer
On Mon, Jan 25, 2016 at 03:41:19PM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > libavcodec/x86/imdct36.asm | 7 +++ > 1 file changed, 7 insertions(+) fate passes with this on mingw32/64 and linux32/64 [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC78