Hi,

2011/10/18 Måns Rullgård <[email protected]>:
> "Ronald S. Bultje" <[email protected]> writes:
>
>> Hi,
>>
>> On Mon, Oct 17, 2011 at 7:52 PM, Ronald S. Bultje <[email protected]> wrote:
>>> ---
>>>  libavcodec/arm/h264dsp_init_arm.c    |    2 +
>>>  libavcodec/h264.c                    |  126 +++++++++------------
>>>  libavcodec/h264dsp.c                 |   28 ++----
>>>  libavcodec/h264dsp.h                 |   10 +-
>>>  libavcodec/h264dsp_template.c        |   28 ++---
>>>  libavcodec/ppc/h264_altivec.c        |   44 +++----
>>>  libavcodec/x86/h264_weight.asm       |  210 
>>> +++++++++++++---------------------
>>>  libavcodec/x86/h264_weight_10bit.asm |  145 ++++++++---------------
>>>  libavcodec/x86/h264dsp_mmx.c         |  175 ++++++++++------------------
>>>  9 files changed, 293 insertions(+), 475 deletions(-)
>>
>> For Mans: I disabled the neon assembly because I got sort of short on
>> time for this one. If you want, I can try trimming it to meet the new
>> API, but it may be faster if you do it. This patch simply disables it.
>> I did update the ppc asm.
>
> Updating the NEON asm is trivial since it already has this structure
> internally.  I'll wait for review of your patch, then prepare an update
> to be folded into the commit.  Sound good?

Absolutely. If you don't have time, I may even have an attempt at it
this weekend.

Ronald
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to