Re: [FFmpeg-devel] [PATCH 0/6] sse2/avx functions for 8-bit simple idct

2017-06-13 Thread Michael Niedermayer
Hi On Tue, Jun 13, 2017 at 03:07:32PM +0200, James Darnley wrote: > On 2017-06-13 00:18, James Darnley wrote: > > On 2017-06-12 18:57, Michael Niedermayer wrote: > >> ./ffplay ~/videos/matrixbench_mpeg2.mpg > >> looks pretty bad > > > > If that would happen to be the FATE sample > > mpeg2/matrixb

Re: [FFmpeg-devel] [PATCH 0/6] sse2/avx functions for 8-bit simple idct

2017-06-13 Thread James Darnley
On 2017-06-13 00:18, James Darnley wrote: > On 2017-06-12 18:57, Michael Niedermayer wrote: >> ./ffplay ~/videos/matrixbench_mpeg2.mpg >> looks pretty bad > > If that would happen to be the FATE sample > mpeg2/matrixbench_mpeg2.lq1.mpg then I see that too. > > As I said on IRC I was able to partl

Re: [FFmpeg-devel] [PATCH 0/6] sse2/avx functions for 8-bit simple idct

2017-06-12 Thread James Darnley
On 2017-06-12 18:57, Michael Niedermayer wrote: > On Mon, Jun 12, 2017 at 03:36:06PM +0200, James Darnley wrote: >> Rounding contributed by Ronald S. Bultje >> --- >> libavcodec/tests/x86/dct.c | 2 ++ >> libavcodec/x86/idctdsp_init.c| 19 +++ >> libavcodec/x86/simple_id

Re: [FFmpeg-devel] [PATCH 0/6] sse2/avx functions for 8-bit simple idct

2017-06-12 Thread Michael Niedermayer
On Mon, Jun 12, 2017 at 03:36:03PM +0200, James Darnley wrote: > I think I have reached the final state for these patches. There has been > little > change to the 1st, 3rd, 4th, and 5th. > > The 2nd adds an option to explicitly control what the macro does after the > IDCT. > This allows the sma