[Mesa-dev] [PATCH] configure.ac: enable_asm=yes when x-compiling across same X86 arch

[dw kim]

Matt, 

I uploaded v2 patch with corrected commit message. Other than 
commit message, everything is still same. Can you please re-review 
it and merge it if there's no issue?

Thanks

On Tue, Feb 16, 2016 at 10:05:24AM -0800, Dongwon Kim wrote:
> Currently, configure script is forcing 'enable_asm' to be 'no'
> whenever cross-compilation is performed on X86 host. This is
> based on an assumption that target architecture is different
> from host's (i.e. ARM). But there's always a case that we do
> cross-compilation for target that is also X86 based just like
> host in which same ASM codes will be supported. 'enable_asm'
> should not be forced to be "no" anymore in this case.
> 
> v2: corrected commit message
> 
> Signed-off-by: Dongwon Kim <dongwon.kim at intel.com>
> ---
>  configure.ac | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index b05f33d..0ad27c9 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -710,8 +710,10 @@ test "x$enable_asm" = xno && AC_MSG_RESULT([no])
>  if test "x$enable_asm" = xyes -a "x$cross_compiling" = xyes; then
>      case "$host_cpu" in
>      i?86 | x86_64 | amd64)
> -        enable_asm=no
> -        AC_MSG_RESULT([no, cross compiling])
> +        if test "x$host_cpu" != "x$target_cpu"; then
> +            enable_asm=no
> +            AC_MSG_RESULT([no, cross compiling])
> +        fi
>          ;;
>      esac
>  fi
> -- 
> 1.9.1
>