Re: [Mesa-dev] [PATCH v3 08/25] configure.ac: Move LLVM version check to the top

2016-10-12 Thread Tobias Droste
Am Mittwoch, 12. Oktober 2016, 20:02:24 CEST schrieb Emil Velikov: > >> With the above > >> Reviewed-by: Emil Velikov > >> > >> > +if test -z "${LLVM_CONFIG}"; then > >> > +if test -n "$llvm_prefix"; then > >> > +AC_PATH_TOOL([LLVM_CONFIG],

Re: [Mesa-dev] [PATCH v3 08/25] configure.ac: Move LLVM version check to the top

2016-10-12 Thread Emil Velikov
On 12 October 2016 at 19:11, Tobias Droste wrote: > Am Mittwoch, 12. Oktober 2016, 10:11:50 CEST schrieb Emil Velikov: >> On 12 October 2016 at 00:02, Tobias Droste wrote: >> > A function with the LLVM version checked is moved to the top. >> > The function is

Re: [Mesa-dev] [PATCH v3 08/25] configure.ac: Move LLVM version check to the top

2016-10-12 Thread Tobias Droste
Am Mittwoch, 12. Oktober 2016, 10:11:50 CEST schrieb Emil Velikov: > On 12 October 2016 at 00:02, Tobias Droste wrote: > > A function with the LLVM version checked is moved to the top. > > The function is called where the old code was. > > Replace the second line with "... in

Re: [Mesa-dev] [PATCH v3 08/25] configure.ac: Move LLVM version check to the top

2016-10-12 Thread Emil Velikov
On 12 October 2016 at 00:02, Tobias Droste wrote: > A function with the LLVM version checked is moved to the top. > The function is called where the old code was. Replace the second line with "... in order to reuse/rework X" > No functional change. > > Signed-off-by: Tobias

[Mesa-dev] [PATCH v3 08/25] configure.ac: Move LLVM version check to the top

2016-10-11 Thread Tobias Droste
A function with the LLVM version checked is moved to the top. The function is called where the old code was. No functional change. Signed-off-by: Tobias Droste --- configure.ac | 91 1 file changed, 49 insertions(+),