On Thu, Jun 7, 2018 at 10:51 AM, Dylan Baker <dy...@pnwbakers.com> wrote:
> Gentoo's ebuild system always adds -m32 to the compiler for doing x86_64
> -> x86 cross builds, while meson expects it not to do that. This
> results in an x86 -> x86 cross build, and assembly gets disabled.
>
> Fixes: 2d62fc06465281d3d45b8a7c7fd2b17ef718448c
>        ("meson: disable x86 asm in fewer cases.")
> CC: Matt Turner <matts...@gmail.com>
> Signed-off-by: Dylan Baker <dylan.c.ba...@intel.com>
> ---
>  meson.build | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/meson.build b/meson.build
> index 32ab30faa4b..e6e2167ebe7 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -852,7 +852,10 @@ if meson.is_cross_build()
>      # TODO: It may be possible to do this with an exe_wrapper (like wine).
>      message('Cross compiling from one OS to another, disabling assembly.')
>      with_asm = false
> -  elif not (build_machine.cpu_family() == 'x86_64' and 
> host_machine.cpu_family() == 'x86')
> +  elif not (build_machine.cpu_family().startswith('x86') and 
> host_machine.cpu_family() == 'x86')
> +    # FIXME: Gentoo always sets -m32 for x86_64 -> x86 builds, resulting in 
> an
> +    # x86 -> x86 cross compile. We use startswith rather than == to handle 
> this
> +    # case.
>      # TODO: There may be other cases where the 64 bit version of the
>      # architecture can run 32 bit binaries (aarch64 and armv7 for example)
>      message('''
> --

Works for me.

Reviewed-by: Matt Turner <matts...@gmail.com>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to