[FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

[Paul B Mahol]

On 8/16/20, Michael Niedermayer <michael at niedermayer.cc> wrote:
> On Sun, Aug 16, 2020 at 08:32:20AM +0200, Paul B Mahol wrote:
>> On 8/16/20, Michael Niedermayer <michael at niedermayer.cc> wrote:
>> > On Sat, Aug 15, 2020 at 07:29:40PM +0200, Paul B Mahol wrote:
>> >> On 8/15/20, Michael Niedermayer <michael at niedermayer.cc> wrote:
>> >> > On Fri, Aug 14, 2020 at 02:24:25PM +0200, Paul B Mahol wrote:
>> >> >> On 8/14/20, Paul B Mahol <onemda at gmail.com> wrote:
>> >> >> > On 8/13/20, Paul B Mahol <onemda at gmail.com> wrote:
>> >> >> >> Hi,
>> >> >> >>
>> >> >> >> patch attached.
>> >> >> >>
>> >> >> >> Please review and/or benchmark, especially .asm file.
>> >> >> >>
>> >> >> >
>> >> >> > Updated patch attached.
>> >> >> >
>> >> >>
>> >> >> Sorry, wrong patch.
>> >> >>
>> >> >> New patch attached.
>> >> >
>> >> >>  Makefile           |    2
>> >> >>  cfhd.c             |  271
>> >> >> +++++++++++------------------------------------------
>> >> >>  cfhd.h             |    3
>> >> >>  cfhddsp.c          |  110 +++++++++++++++++++++
>> >> >>  cfhddsp.h          |   42 ++++++++
>> >> >>  x86/Makefile       |    2
>> >> >>  x86/cfhddsp.asm    |  227
>> >> >> ++++++++++++++++++++++++++++++++++++++++++++
>> >> >>  x86/cfhddsp_init.c |   44 ++++++++
>> >> >>  8 files changed, 488 insertions(+), 213 deletions(-)
>> >> >> 9312085fc5971774cdc8511019fa4023cf269100
>> >> >> 0001-avcodec-cfhd-add-x86-SIMD.patch
>> >> >> From 874fd9e604a6dcd55cca77c7256a633e5739da77 Mon Sep 17 00:00:00
>> >> >> 2001
>> >> >> From: Paul B Mahol <onemda at gmail.com>
>> >> >> Date: Sun, 9 Aug 2020 17:47:34 +0200
>> >> >> Subject: [PATCH] avcodec/cfhd: add x86 SIMD
>> >> >
>> >> > This changes the fate checksums here
>> >> >
>> >>
>> >> Are you sure you tested latest patch?
>> >
>> > iam sure i tested the patch i replied to. I assumed that to be the
>> > latest
>> >
>>
>> What operating system, 32bit or 64bit?
>
> Ubuntu 18.04.3 LTS x86-64
> gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0

Try latest patch instead.
Previous one could not work on 64bit.

>
>
>
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> The real ebay dictionary, page 1
> "Used only once"    - "Some unspecified defect prevented a second use"
> "In good condition" - "Can be repaird by experienced expert"
> "As is" - "You wouldnt want it even if you were payed for it, if you knew
> ..."
>