[Mesa-stable] [PATCH 6/7] configure.ac: add missing LLVM components for OpenCL
Andres Gomez
agomez at igalia.com
Wed Oct 25 22:38:53 UTC 2017
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.velikov at 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-stable at lists.freedesktop.org
> Signed-off-by: Emil Velikov <emil.velikov at 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
More information about the mesa-stable
mailing list