[Ffmpeg-cvslog] r7994 - trunk/libavcodec/i386/h264dsp_mmx.c

Ivan Kalvachev ikalvachev
Sat Feb 17 23:09:25 CET 2007


2007/2/16, michael <subversion at mplayerhq.hu>:
> Author: michael
> Date: Fri Feb 16 22:22:53 2007
> New Revision: 7994
>
> Modified:
>    trunk/libavcodec/i386/h264dsp_mmx.c
>
> Log:
> add a few asserts to ensure alignment
>
>
> Modified: trunk/libavcodec/i386/h264dsp_mmx.c
> ==============================================================================
> --- trunk/libavcodec/i386/h264dsp_mmx.c (original)
> +++ trunk/libavcodec/i386/h264dsp_mmx.c Fri Feb 16 22:22:53 2007
> @@ -1311,6 +1311,7 @@
>      uint64_t temp[SIZE*(SIZE<8?12:24)/4 + SIZE*SIZE/8];\
>      uint8_t * const halfHV= (uint8_t*)temp;\
>      int16_t * const tmp= ((int16_t*)temp) + SIZE*SIZE/2;\
> +    assert((int)temp & 7 == 0);\

Why release quality code should abort() on none-SSE cpu because of
unaligned MMX access?

You can simply define and use aligned temp[] array in the dspcontext!




More information about the ffmpeg-cvslog mailing list