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....@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 > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
