this patch should be not part of this serries and can go to master as
standalone patch. No need of camel case is shor description. Long
description in git comment is required.

Maxim.


On 04/04/17 21:47, Brian Brooks wrote:
> Signed-off-by: Brian Brooks <brian.bro...@arm.com>
> ---
>  configure.ac | 30 ++++++++++++++++--------------
>  1 file changed, 16 insertions(+), 14 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 419c136f..da091ab1 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -304,20 +304,22 @@ ODP_CFLAGS="$ODP_CFLAGS -std=c99"
>  # Extra flags for example to suppress certain warning types
>  ODP_CFLAGS="$ODP_CFLAGS $ODP_CFLAGS_EXTRA"
>  
> -#########################################################################
> -# Check if compiler supports cmpxchng16
> -##########################################################################
> -if test "${CC}" != "gcc" -o ${CC_VERSION_MAJOR} -ge 5; then
> -   my_save_cflags="$CFLAGS"
> -
> -   CFLAGS=-mcx16
> -   AC_MSG_CHECKING([whether CC supports -mcx16])
> -   AC_COMPILE_IFELSE([AC_LANG_PROGRAM([])],
> -         [AC_MSG_RESULT([yes])]
> -         [ODP_CFLAGS="$ODP_CFLAGS $CFLAGS"],
> -         [AC_MSG_RESULT([no])]
> -         )
> -   CFLAGS="$my_save_cflags"
> +##########################################################################
> +# Check if compiler supports cmpxchng16 on x86-based architectures
> +##########################################################################
> +if "${host}" == i?86* -o "${host}" == x86*; then
> +  if test "${CC}" != "gcc" -o ${CC_VERSION_MAJOR} -ge 5; then
> +     my_save_cflags="$CFLAGS"
> +
> +     CFLAGS=-mcx16
> +     AC_MSG_CHECKING([whether CC supports -mcx16])
> +     AC_COMPILE_IFELSE([AC_LANG_PROGRAM([])],
> +         [AC_MSG_RESULT([yes])]
> +         [ODP_CFLAGS="$ODP_CFLAGS $CFLAGS"],
> +         [AC_MSG_RESULT([no])]
> +         )
> +     CFLAGS="$my_save_cflags"
> +  fi
>  fi
>  
>  ##########################################################################
> 

Reply via email to