[FFmpeg-devel] [PATCH 10/11] avcodec/x86: add an 8-bit simple IDCT function based on the x86-64 high depth functions

Michael Niedermayer michael at niedermayer.cc
Wed Jun 21 03:33:19 EEST 2017


On Tue, Jun 20, 2017 at 08:24:55PM -0400, Ronald S. Bultje wrote:
> Hi,
> 
> On Tue, Jun 20, 2017 at 7:13 PM, Michael Niedermayer <michael at niedermayer.cc
> > wrote:
> 
> > On Mon, Jun 19, 2017 at 05:11:03PM +0200, James Darnley wrote:
> > > Includes add/put functions
> > >
> > > Rounding contributed by Ronald S. Bultje
> > > ---
> > >  libavcodec/tests/x86/dct.c                |  2 +
> > >  libavcodec/x86/idctdsp_init.c             | 23 ++++++++
> > >  libavcodec/x86/simple_idct.h              |  9 +++
> > >  libavcodec/x86/simple_idct10.asm          | 92
> > +++++++++++++++++++++++++++++++
> > >  libavcodec/x86/simple_idct10_template.asm |  6 +-
> > >  5 files changed, 130 insertions(+), 2 deletions(-)
> >
> > breaks:
> >
> > ./ffmpeg -i matrixbench_mpeg2.mpg -an -vframes 1  -vf
> > format=gbrp,spp=1:63,format=gbrp -qscale 1 file.avi
> 
> 
> Please define "breaks".

it looked like the blocks where somehow using the wrong scantable
that is recognizable but clearly not correct looking

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

While the State exists there can be no freedom; when there is freedom there
will be no State. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170621/6aff2dc8/attachment.sig>


More information about the ffmpeg-devel mailing list