[FFmpeg-devel] [PATCH 1/2] lavc: define STRIDE_ALIGN as 32 when compiling with AVX support

[Ronald S. Bultje]

Hi,

On Sun, May 18, 2014 at 8:59 PM, James Almer <jamrial at gmail.com> wrote:

> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
>  libavcodec/internal.h | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/libavcodec/internal.h b/libavcodec/internal.h
> index 603ffcf..948d006 100644
> --- a/libavcodec/internal.h
> +++ b/libavcodec/internal.h
> @@ -35,7 +35,9 @@
>
>  #define FF_SANE_NB_CHANNELS 63U
>
> -#if HAVE_NEON || ARCH_PPC || HAVE_MMX
> +#if HAVE_AVX
> +#   define STRIDE_ALIGN 32
> +#elif HAVE_NEON || ARCH_PPC || HAVE_MMX
>  #   define STRIDE_ALIGN 16
>  #else
>  #   define STRIDE_ALIGN 8


I'll be the first to admit this isn't very pretty, but it does the job and
I can't think of anything better that doesn't add runtime constraints. So
if others don't have strong objections or better ideas, I'd like this to be
committed. It'll help a lot for future AVX/2 optimizations.

Ronald