On Sun, Oct 20, 2013 at 10:28:38AM -0500, Kieran Kunhya wrote:
> --- a/libavutil/cpu.h
> +++ b/libavutil/cpu.h
> @@ -48,6 +48,8 @@
> #define AV_CPU_FLAG_XOP 0x0400 ///< Bulldozer XOP functions
> #define AV_CPU_FLAG_FMA4 0x0800 ///< Bulldozer FMA4 functions
> #define AV_CPU_FLAG_CMOV 0x1000 ///< i686 cmov
> +#define AV_CPU_FLAG_AVX2 0x8000 ///< AVX2 functions: requires OS
> support even if YMM registers aren't used
> +
stray extra empty line
> --- a/libavutil/x86/cpu.c
> +++ b/libavutil/x86/cpu.c
> @@ -134,6 +134,13 @@ int ff_get_cpu_flags_x86(void)
> if ((eax & 0x6) == 0x6)
> rval |= AV_CPU_FLAG_AVX;
> }
> + if (HAVE_AVX2 && max_std_level >= 7)
> + {
Wrong { placement, please follow the style of the surrounding code,
not x264's.
Why did you switch away from the ifdef?
Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
