[FFmpeg-devel] [PATCH] avcodec/x86/mpegvideoenc: remove av_assert2() for variable alignment

Ramiro Polla ramiro.polla at gmail.com
Thu Aug 22 04:05:23 EEST 2024


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 eedbd752ac..3cc322b10e 100644
--- a/libavcodec/x86/mpegvideoenc_template.c
+++ b/libavcodec/x86/mpegvideoenc_template.c
@@ -101,8 +101,6 @@ static int RENAME(dct_quantize)(MpegEncContext *s,
     const uint8_t *scantable;
     const uint16_t *inv_scantable_p1;
 
-    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 ...
 
-- 
2.39.2



More information about the ffmpeg-devel mailing list