[FFmpeg-cvslog] Align stack variables manually in yadif x86 code, should fix ticket25

Aurelien Jacobs aurel at gnuage.org
Wed Apr 13 00:02:33 CEST 2011


On Thu, Apr 07, 2011 at 01:49:39AM +0200, Michael Niedermayer wrote:
> ffmpeg | branch: master | Michael Niedermayer <michaelni at gmx.at> | Thu Apr  7 00:31:21 2011 +0200| [4b9c03b7f08f51fea3c8719f169368a46a0d3b63] | committer: Michael Niedermayer
> 
> Align stack variables manually in yadif x86 code, should fix ticket25
> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> 
> [...]
> 
> +    uint8_t tmp[5*16];
> +    uint8_t *tmpA= (uint8_t*)(((uint64_t)(tmp+15)) & ~15);

What about using FFALIGN() here ? It still requires the cast so it's not
a big improvement, but still.
See attached patch.

Aurel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-yadif-use-FFALIGN-instead-of-reimplementing-it.patch
Type: text/x-diff
Size: 903 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-cvslog/attachments/20110413/67b69c65/attachment.bin>


More information about the ffmpeg-cvslog mailing list