Re: [PATCH] fix broken build since r38235

Reimar Döffinger <[email protected]> Sun, 24 Jan 2021 13:44:54 +0100
Newsgroups gmane.comp.video.mplayer.devel
Message-ID <YA1rxgV6XDeaL/[email protected]>
On Sat, Jan 23, 2021 at 11:56:34PM +0100, Erik Auerswald wrote:
> Hi,
>
> On 23.01.21 23:28, Reimar Döffinger wrote:
> > > Using DECLARE_ALIGNED instead of LOCAL_ALIGNED_16 fixes the
> > > compilation error for me, please see attached patch.  I hope
> > > that I've used DECLARE_ALIGNED correctly.
> >
> > Using DECLARE_ALIGNED is not correct to use for stack variables.
>
> Thanks, I did not know that (and did not investigate, my fault).

That's fine, we in fact had some code using DECLARE_ALIGNED for stack
variables and in practice it seems to have worked anyway.
So your solution was not entirely wrong, just not quite right.
_______________________________________________
MPlayer-dev-eng mailing list
[email protected]
https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng