[FFmpeg-devel] [PATCH 1/2] swresample: Refactor resample asm and port it to yasm
James Almer
jamrial at gmail.com
Wed Mar 19 22:39:59 CET 2014
On 19/03/14 6:02 PM, Michael Niedermayer wrote:
> On Wed, Mar 19, 2014 at 02:49:33AM -0300, James Almer wrote:
>> This reduces code duplication and makes it easier to implement new asm
>> functions in the future
>>
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
>> libswresample/resample.c | 96 ++++++++++---------------------------
>> libswresample/resample_template.c | 49 +++++++------------
>> libswresample/swresample_internal.h | 24 ++++++++++
>> libswresample/x86/Makefile | 1 +
>> libswresample/x86/resample.asm | 64 +++++++++++++++++++++++++
>> libswresample/x86/resample_mmx.h | 74 ----------------------------
>> libswresample/x86/swresample_x86.c | 16 +++++++
>> 7 files changed, 148 insertions(+), 176 deletions(-)
>> create mode 100644 libswresample/x86/resample.asm
>> delete mode 100644 libswresample/x86/resample_mmx.h
>
> this seems not fully working
> for example -af aformat=s32,aresample=48000,aformat=s32 (with 44100 input)
> seems failing
Yeah, just found what was wrong. I accidentally removed the cast to FELEM2 for
the filter values in the c version of the scalarproduct function when i moved it.
Oddly enough fate-swr didn't trigger this. Maybe it should be expanded?
Will send a new patch in a moment. Thanks.
More information about the ffmpeg-devel
mailing list