[FFmpeg-devel] [PATCH 0/5] huffyuv and lossless_video dsp contexts reorganization

[James Almer]

On 1/12/2017 2:54 PM, Paul B Mahol wrote:
> On 1/12/17, James Almer <jamrial at gmail.com> wrote:
>> On 1/8/2017 12:25 PM, James Almer wrote:
>>> This moves code around so dsp functions used by either one or more
>>> codecs are in the corresponding contexts. Decoder and encoder dsp
>>> functions are also now properly split.
>>>
>>> James Almer (5):
>>>   lossless_videodsp: move shared functions from huffyuvdsp
>>>   huffyuvencdsp: move functions only used by huffyuv from
>>> lossless_videodsp
>>>   huffyuvencdsp: move shared functions to a new lossless_videoencdsp
>>> context
>>>   huffyuvdsp: move functions only used by huffyuv from lossless_videodsp
>>>   lossless_videodsp: rename add_hfyu_left_pred_int16 to
>>> add_left_pred_int16
>>>
>>>  configure                                          |  17 +-
>>>  libavcodec/Makefile                                |   1 +
>>>  libavcodec/huffyuv.c                               |   1 -
>>>  libavcodec/huffyuv.h                               |   2 +
>>>  libavcodec/huffyuvdec.c                            |  72 ++---
>>>  libavcodec/huffyuvdsp.c                            |  65 ++--
>>>  libavcodec/huffyuvdsp.h                            |  20 +-
>>>  libavcodec/huffyuvenc.c                            |  46 +--
>>>  libavcodec/huffyuvencdsp.c                         |  69 ++--
>>>  libavcodec/huffyuvencdsp.h                         |  25 +-
>>>  libavcodec/lagarith.c                              |  12 +-
>>>  libavcodec/lossless_videodsp.c                     | 100 +++---
>>>  libavcodec/lossless_videodsp.h                     |  19 +-
>>>  libavcodec/lossless_videoencdsp.c                  |  84 +++++
>>>  libavcodec/lossless_videoencdsp.h                  |  41 +++
>>>  libavcodec/magicyuv.c                              |  38 ++-
>>>  libavcodec/pngenc.c                                |  10 +-
>>>  libavcodec/ppc/Makefile                            |   2 +-
>>>  ...uvdsp_altivec.c => lossless_videodsp_altivec.c} |   4 +-
>>>  libavcodec/utvideo.h                               |   8 +-
>>>  libavcodec/utvideodec.c                            |  16 +-
>>>  libavcodec/utvideoenc.c                            |   5 +-
>>>  libavcodec/vble.c                                  |  10 +-
>>>  libavcodec/x86/Makefile                            |   4 +-
>>>  libavcodec/x86/huffyuvdsp.asm                      | 291
>>> +++++++----------
>>>  libavcodec/x86/huffyuvdsp_init.c                   |  82 +----
>>>  libavcodec/x86/huffyuvencdsp.asm                   | 211 ++++++-------
>>>  libavcodec/x86/huffyuvencdsp_init.c                |  54 ++++
>>>  libavcodec/x86/lossless_videodsp.asm               | 348
>>> ++++++++++-----------
>>>  libavcodec/x86/lossless_videodsp_init.c            |  99 ++++--
>>>  libavcodec/x86/lossless_videoencdsp.asm            | 150 +++++++++
>>>  ...yuvencdsp_mmx.c => lossless_videoencdsp_init.c} |  14 +-
>>>  32 files changed, 1068 insertions(+), 852 deletions(-)
>>>  create mode 100644 libavcodec/lossless_videoencdsp.c
>>>  create mode 100644 libavcodec/lossless_videoencdsp.h
>>>  rename libavcodec/ppc/{huffyuvdsp_altivec.c =>
>>> lossless_videodsp_altivec.c} (95%)
>>>  create mode 100644 libavcodec/x86/huffyuvencdsp_init.c
>>>  create mode 100644 libavcodec/x86/lossless_videoencdsp.asm
>>>  rename libavcodec/x86/{huffyuvencdsp_mmx.c =>
>>> lossless_videoencdsp_init.c} (88%)
>>>
>>
>> Ping for patchset.
> 
> This just moves lines arround, I'm fine with it if fate passes.

Pushed then. Fate will tell if ppc was broken or not.

You can resend your magicyuv simd patch using its own context now.