From b6f7271fa914e425142023a7cedff682183a6bf8 Mon Sep 17 00:00:00 2001 From: Ramiro Polla Date: Thu, 22 Aug 2024 03:05:23 +0200 Subject: [PATCH] avcodec/x86/mpegvideoenc: remove av_assert2() for variable alignment It's safe to assume that LOCAL_ALIGNED_16 does indeed align. Otherwise we would have many more problems... This assert was added in f8188626 all the way back in 2003. --- libavcodec/x86/mpegvideoenc_template.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavcodec/x86/mpegvideoenc_template.c b/libavcodec/x86/mpegvideoenc_template.c index 5f013be7f5..4096d6391f 100644 --- a/libavcodec/x86/mpegvideoenc_template.c +++ b/libavcodec/x86/mpegvideoenc_template.c @@ -99,8 +99,6 @@ static int RENAME(dct_quantize)(MpegEncContext *s, const uint16_t *qmat, *bias; LOCAL_ALIGNED_16(int16_t, temp_block, [64]); - av_assert2((7&(uintptr_t)(&temp_block[0])) == 0); //did gcc align it correctly? - //s->fdct (block); RENAME_FDCT(ff_fdct)(block); // cannot be anything else ...