[FFmpeg-devel] [PATCH] Patch cleanup for MPEG 1 & 2 optimizations

Michael Niedermayer michaelni
Wed Apr 16 23:23:00 CEST 2008


On Wed, Apr 16, 2008 at 10:31:34PM +0200, Keiji Costantini wrote:
> Benoit Fouet ha scritto:
>> Hi,
>>
>> strites wrote:
>>   
>>> diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c
>>> index c128949..ba0e73b 100644
>>> --- a/libavcodec/mpegvideo.c
>>> +++ b/libavcodec/mpegvideo.c
>>> @@ -1978,6 +1978,7 @@ skip_idct:
>>>      }
>>>  }
>>>
>>> +av_always_inline
>>>  void MPV_decode_mb(MpegEncContext *s, DCTELEM block[12][64]){
>>>       
>>
>> what exactly is the point of this hunk ??
>>
>>   
> Ups...
>
> I corrected that and got actually a speed gain.

Your patches still contain the typo jeff pointed out, also i think
they are identical to jeffs variant after the typo is fixed.

The question i have is the code with your patches applied as fast as
if is_mpeg12 would be hardcoded to 1 ?

Jeffs benchmarks indicated that there still was a speed difference left
but it could be as well random variation. If there is still a difference
left i would like to know what is causing this difference. Is it just
the overhead for the extra if() and call or is there more?

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Good people do not need laws to tell them to act responsibly, while bad
people will find a way around the laws. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080416/3e74eaba/attachment.pgp>



More information about the ffmpeg-devel mailing list