On Wed, 29 Mar 2017, 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/.
---
libavutil/internal.h | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
FWIW, I had to amend this patch to include mem.h in internal.h, as it uses
DECLARE_ALIGNED which is defined in mem.h. (The same was present in the
alternative patch as well, but I didn't notice the issue so far.)
// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
