On 21/07/14 6:21 PM, Diego Biurrun wrote:
> ---
> 
> I'm slightly unsure about this one.  MMX does not require 16-byte aligned
> buffers, nor does PowerPC IIRC, but SSE and AltiVec do, so I believe my
> solution is closer to the original intention.  Please do correct me if I
> am wrong...
> 
>  configure          | 2 ++
>  libavcodec/utils.c | 2 +-
>  2 files changed, 3 insertions(+), 1 deletion(-)
> 

[...]

> diff --git a/libavcodec/utils.c b/libavcodec/utils.c
> index 2abc376..16c30c3 100644
> --- a/libavcodec/utils.c
> +++ b/libavcodec/utils.c
> @@ -189,7 +189,7 @@ int ff_side_data_update_matrix_encoding(AVFrame *frame,
>      return 0;
>  }
>  
> -#if HAVE_NEON || ARCH_PPC || HAVE_MMX
> +#if HAVE_SIMD_ALIGN_16
>  #   define STRIDE_ALIGN 16
>  #else
>  #   define STRIDE_ALIGN 8

#if HAVE_NEON || HAVE_ALTIVEC || HAVE_SSE
#   define STRIDE_ALIGN 16
#else
#   define STRIDE_ALIGN 8
#endif

is simpler than adding another HAVE_ define for this single use.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to