[Ffmpeg-devel] [PATCH] h264 *_pixels16_l2_altivec

Loren Merritt lorenm
Tue Mar 7 22:52:15 CET 2006


On Tue, 7 Mar 2006, Luca Barbato wrote:
> Romain Dolbeau wrote:
>>
>> I think the fact that src_stride2 is hardwired to 16
>> should be mentioned above the functions, should it
>> be called from someplace that doesn't know about
>> the restriction.
>
> The previous code assumed the same, so I just removed the define and
> hardwired it there.
>
>> Doesn't H264 requires image_stride % 16 == 0 ? Because
>> if it can be assumed src_stride1 % 16 == 0, then the
>> vec_lvsl can be moved out of the loop.
>
> Not sure I check this evening
>
>> (I don't think that src1/src2/dst are always properly
>> aligned ? that could solve the performance problem)
>
> sadly they are all unaligned.

dst and one of the src can be made aligned.

--Loren Merritt





More information about the ffmpeg-devel mailing list