[Ffmpeg-devel] [PATCH] Snow mc_block mmx optimization

Guillaume POIRIER gpoirier
Fri Mar 24 21:27:49 CET 2006


Guillaume POIRIER wrote:
> Hi,
> Oded Shimon wrote:
>> On Fri, Mar 24, 2006 at 07:52:01PM +0100, Michael Niedermayer wrote:
>>> Hi
>>>
>>> On Fri, Mar 24, 2006 at 04:18:04PM +0200, Oded Shimon wrote:
>>>> On Thu, Mar 23, 2006 at 09:43:29PM +0100, Guillaume POIRIER wrote:
> 
>>>> New patch. Also has the _mmx function names you wanted.
>>> patch ok IMHO if regression tests pass, except that the mmx stuff should
>>> go into snowdsp_mmx.c
>>
>> Any suggestion to do this without exporting 16 mc_block functions?.. 
>> Or is this fine and just do this?
>>
>> extern void ff_snow_mc_block_x0_mmx();
>> extern void ff_snow_mc_block_x2_mmx();
> 
> Attached is the patch that actually builds, works on AMD64, and passes 
> regression tests.

Ahem, scratch that one: it features a change of libavformat/avienc.c I 
had on my tree. Oded's patch is the same as "mine", but doesn"t have 
that stupid mistake in it.

Guillaume





More information about the ffmpeg-devel mailing list