[FFmpeg-devel] [PATCH] ff_scalarproduct_float_sse

Måns Rullgård mans
Wed Jan 20 15:48:57 CET 2010


Michael Niedermayer <michaelni at gmx.at> writes:

> On Tue, Jan 19, 2010 at 11:42:40PM -0500, Alex Converse wrote:
>> This cause a >50% decrease in SBR decode time.
>> 
>> For the time being it can help in the other places where
>> scalarproduct_float() is used.
>> 
>> Regards,
>> Alex Converse
>
>>  dsputil_mmx.c    |    5 +++++
>>  dsputil_yasm.asm |   25 +++++++++++++++++++++++++
>
> Would you mind to avoid yasm and use gcc asm instead ?
>
> I have no problem with yasm as such but gcc asm is more portable and
> can be integrated with C code if we ever want that.

I have to disagree.  Just look at how many FATE targets broke with
your change to h264_loop_filter_strength_mmx2 yesterday.  Several
compilers are still failing to build it.

I'm not aware of any serious OS on which yasm doesn't run, so the
portability argument doesn't hold water.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list