Emil, by 17.2, Clover is still asking only for LLVM 3.6 and "coverage" doesn't seem to be there.
Hence, I think we should drop or backport this patch. In any case, check my other mail about this patch series ... On Thu, 2017-10-05 at 11:19 +0100, Emil Velikov wrote: > From: Emil Velikov <emil.veli...@collabora.com> > > Coverage and LTO seems to be hard requirements for Clang, while > coroutines is needed as of LLVM/Clang 4.0. > > Mark the last one as "optional" so we handle every case. > > Cc: mesa-sta...@lists.freedesktop.org > Signed-off-by: Emil Velikov <emil.veli...@collabora.com> > --- > configure.ac | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/configure.ac b/configure.ac > index 56cbf26c778..df8cfa6b71b 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -2300,13 +2300,16 @@ if test "x$enable_opencl" = xyes; then > > llvm_add_default_components "opencl" > llvm_add_component "all-targets" "opencl" > + llvm_add_component "coverage" "opencl" > llvm_add_component "linker" "opencl" > llvm_add_component "instrumentation" "opencl" > llvm_add_component "ipo" "opencl" > llvm_add_component "irreader" "opencl" > + llvm_add_component "lto" "opencl" > llvm_add_component "option" "opencl" > llvm_add_component "objcarcopts" "opencl" > llvm_add_component "profiledata" "opencl" > + llvm_add_optional_component "coroutines" "opencl" > > dnl Check for Clang internal headers > if test -z "$CLANG_LIBDIR"; then -- Br, Andres _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
