[FFmpeg-devel] [PATCH] ALS: Solve Issue 1657

Thilo Borgmann thilo.borgmann
Thu Jan 7 13:15:20 CET 2010


Am 07.01.10 09:33, schrieb Reimar D?ffinger:
> On Thu, Jan 07, 2010 at 01:19:57AM +0100, Thilo Borgmann wrote:
>>>>>            y += MUL64(lpc_cof[sb], raw_samples[sb]);
>>>>>
>>>>>        *raw_samples++ -= y >> 20;
>>>>>    }
>>>
>>> checking the end via raw_samples < end should be faster
>>> (1 variable less, 1 ++ less)
>>> also if this is on x86_64 try to make sb long instead of int this
>>> too might help
>>
>> Yes and yes:
> 
> x86_reg would also work for mingw64

Patch attached using faster "raw_samples < end"-loop.

What still might be improved, is to type variable "sb" for different
architectures but my question about how to do this in ffmpeg still
remains open. Though, this might be a more general request for the
decoder and therefore worth another patch/thread...

-Thilo
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: als_issue1657_better.rev1.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100107/281c4584/attachment.txt>



More information about the ffmpeg-devel mailing list