Re: [FFmpeg-devel] [PATCH 7/9] hevc: Fix aligned array declarations
On 19/10/17 00:43, Michael Niedermayer wrote: > On Tue, Oct 17, 2017 at 10:12:23PM +0100, Mark Thompson wrote: >> (cherry picked from commit d41e10c1485ec34aa342f7bc2e5bf4f9b6e66414) >> --- >> This and the following patches were found with the TI ARM compiler. >> >> >> libavcodec/hevcdsp.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) > > ok > > also theres a > DECLARE_ALIGNED(32, int16_t, tmp [MAX_PB_SIZE * MAX_PB_SIZE]); > in libavcodec/hevcdec.h > > which seems to have the same issue The TI compiler doesn't seem to mind that one (single-dimensional?), but I'll fix it anyway in the same patch in case something else does. A bit of grep finds some more cases in libavcodec/mips/*_mmi.c and libavutil/lls.h, I'll send some more patches for those later. Thanks, - Mark ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH 7/9] hevc: Fix aligned array declarations
On Tue, Oct 17, 2017 at 10:12:23PM +0100, Mark Thompson wrote: > (cherry picked from commit d41e10c1485ec34aa342f7bc2e5bf4f9b6e66414) > --- > This and the following patches were found with the TI ARM compiler. > > > libavcodec/hevcdsp.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) ok also theres a DECLARE_ALIGNED(32, int16_t, tmp [MAX_PB_SIZE * MAX_PB_SIZE]); in libavcodec/hevcdec.h which seems to have the same issue [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Concerning the gods, I have no means of knowing whether they exist or not or of what sort they may be, because of the obscurity of the subject, and the brevity of human life -- Protagoras signature.asc Description: Digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel