Re: [Mesa-dev] [PATCH v3 12/25] configure.ac: Move gallium LLVM checks

2016-10-12 Thread Tobias Droste
Am Mittwoch, 12. Oktober 2016, 19:39:47 CEST schrieb Jose Fonseca: > >> > >> Right now --enable-gallium-llvm will be set to no if it is "auto" on non > >> x86. > >> > >> gallivm/llvm use the x86 target > > > > I haven't look too closely at the llvm code but there's nothing in > > scons/automake

Re: [Mesa-dev] [PATCH v3 12/25] configure.ac: Move gallium LLVM checks

2016-10-12 Thread Jose Fonseca
On 12/10/16 19:30, Emil Velikov wrote: On 12 October 2016 at 19:14, Tobias Droste wrote: Am Mittwoch, 12. Oktober 2016, 10:42:19 CEST schrieb Emil Velikov: On 12 October 2016 at 00:02, Tobias Droste wrote: gallium_require_llvm() { -if test "x$MESA_LLVM" = x0; then -case "$host

Re: [Mesa-dev] [PATCH v3 12/25] configure.ac: Move gallium LLVM checks

2016-10-12 Thread Tobias Droste
Am Mittwoch, 12. Oktober 2016, 19:30:51 CEST schrieb Emil Velikov: > >> > >> The function it quite "ugly" as-is and this patch changes things in a fun > >> way. > >> > >> Namely: before you'll get the minimum required version check > >> regardless of host_cpu for everyone, while now you get the o

Re: [Mesa-dev] [PATCH v3 12/25] configure.ac: Move gallium LLVM checks

2016-10-12 Thread Emil Velikov
On 12 October 2016 at 19:14, Tobias Droste wrote: > Am Mittwoch, 12. Oktober 2016, 10:42:19 CEST schrieb Emil Velikov: >> On 12 October 2016 at 00:02, Tobias Droste wrote: >> > >> > gallium_require_llvm() { >> > >> > -if test "x$MESA_LLVM" = x0; then >> > -case "$host" in *gnux32) re

Re: [Mesa-dev] [PATCH v3 12/25] configure.ac: Move gallium LLVM checks

2016-10-12 Thread Tobias Droste
Am Mittwoch, 12. Oktober 2016, 10:42:19 CEST schrieb Emil Velikov: > On 12 October 2016 at 00:02, Tobias Droste wrote: > > > > gallium_require_llvm() { > > > > -if test "x$MESA_LLVM" = x0; then > > -case "$host" in *gnux32) return;; esac > > -case "$host_cpu" in > > -

Re: [Mesa-dev] [PATCH v3 12/25] configure.ac: Move gallium LLVM checks

2016-10-12 Thread Emil Velikov
On 12 October 2016 at 00:02, Tobias Droste wrote: > There's already a function for gallium LLVM checks. > Move the remaining code to this function and call it for every driver > that uses LLVM. > > Signed-off-by: Tobias Droste > --- > configure.ac | 31 +-- > 1 file c

[Mesa-dev] [PATCH v3 12/25] configure.ac: Move gallium LLVM checks

2016-10-11 Thread Tobias Droste
There's already a function for gallium LLVM checks. Move the remaining code to this function and call it for every driver that uses LLVM. Signed-off-by: Tobias Droste --- configure.ac | 31 +-- 1 file changed, 13 insertions(+), 18 deletions(-) diff --git a/configure.