[FFmpeg-devel] [PATCH] h264 luma interpolation 8x8 for altivec

[Guillaume Poirier]

Hi,

Luca Barbato wrote:
> Guillaume POIRIER wrote:
>> Part 3 doesn't apply anymore, here's an updated patch that applies to
>> current GIT tree ,-)
>>
> 
> Could you apply the SIZE -> 16 separately or just remove it?


Yes, I can apply it separately, but not remove it.

Quoting Mauricio Alvarez:
"Well, the possible values of SIZE are 16 and 8 (for Altivec), but in
both cases we have to perform loads (and stores) in the temporary
arrays of 16 bytes.
Because of that I replaced uint8_t, half[SIZE*SIZE] by uint8_t,
half[16*16]. In this way we avoid to perform extra alignment checks in
the 8x8 case."


This seems like a reasonable reason to do the SIZE -> 16, don't you think?

Guillaume