[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