On Wed, Mar 29, 2017 at 02:02:32PM +0300, Martin Storsjö wrote: > Previously, the former form always produced a manually aligned, > padded buffer, while the latter can use DECLARE_ALIGNED, if that > amount of stack alignment is supported. > --- > The LOCAL_ALIGNED(4) case in hevc_mvs.c had to be removed, since we > don't have any LOCAL_ALIGNED_4 macro. > > This avoids having to use the slightly awkward LOCAL_ALIGNED_xx syntax. > Contrary to the patches posted yesterday, this leaves the source > inconsistent, using both syntaxes though. Should we settle on using > LOCAL_ALIGNED(xx) consistently instead? > > That is, this is an alternative to https://patches.libav.org/patch/63160/.
I prefer this approach and I think we should settle on LOCAL_ALIGNED(xx). OK Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
