[FFmpeg-devel] [PATCH] speedhq: Align blocks variable properly.

James Almer jamrial at gmail.com
Fri Jan 13 21:51:47 EET 2017


On 1/13/2017 3:04 PM, Steinar H. Gunderson wrote:
> Seemingly ff_clear_block_sse assumed that the block array is aligned,
> so make sure it is.

And so does IDCTDSPContext.idct_put()

> ---
>  libavcodec/speedhq.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/speedhq.c b/libavcodec/speedhq.c
> index 9c21c0b1be..30160dd3f2 100644
> --- a/libavcodec/speedhq.c
> +++ b/libavcodec/speedhq.c
> @@ -224,7 +224,7 @@ static inline int decode_dct_block(const SHQContext *s, GetBitContext *gb, int l
>  {
>      const int *quant_matrix = s->quant_matrix;
>      const uint8_t *scantable = s->intra_scantable.permutated;
> -    int16_t block[64];
> +    DECLARE_ALIGNED(16, int16_t, block)[64];

Changed to use LOCAL_ALIGNED_16() instead and pushed.

Thanks.



More information about the ffmpeg-devel mailing list