[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