Re: [FFmpeg-devel] [PATCH 2/3] simple_idct12: align C and x86

2015-10-14 Thread Michael Niedermayer
On Wed, Oct 14, 2015 at 08:46:22AM +0200, Christophe Gisquet wrote: > Hi, > > 2015-10-14 0:04 GMT+02:00 Michael Niedermayer : > > the ome and syserr values worsen by this > > I have mixed feelings about this, too. > > On the one hand, omse in any case says anyway none of

Re: [FFmpeg-devel] [PATCH 2/3] simple_idct12: align C and x86

2015-10-13 Thread Michael Niedermayer
On Tue, Oct 13, 2015 at 09:21:40PM +0200, Christophe Gisquet wrote: > Results for omse on the 3 idct dct-test. > > C: 0.16915859 0.11848359 0.12913125 > x86: 0.16883281 0.11849063 0.19041875 > > Using 14 and 17 as shifts subtantially improve those, but actually > cause overflows and

Re: [FFmpeg-devel] [PATCH 2/3] simple_idct12: align C and x86

2015-10-13 Thread Ganesh Ajjanagadde
On Tue, Oct 13, 2015 at 6:04 PM, Michael Niedermayer wrote: > On Tue, Oct 13, 2015 at 09:21:40PM +0200, Christophe Gisquet wrote: >> Results for omse on the 3 idct dct-test. >> >> C: 0.16915859 0.11848359 0.12913125 >> x86: 0.16883281 0.11849063 0.19041875 >> >>