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

Ramiro Polla git at videolan.org
Tue Sep 3 18:06:50 EEST 2024


ffmpeg | branch: master | Ramiro Polla <ramiro.polla at gmail.com> | Thu Aug 22 03:05:23 2024 +0200| [b6f7271fa914e425142023a7cedff682183a6bf8] | committer: Ramiro Polla

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.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=b6f7271fa914e425142023a7cedff682183a6bf8
---

 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 ...
 



More information about the ffmpeg-cvslog mailing list