[Mesa-dev] [PATCH 1/7] configure.ac: remove no longer necessary llvm-config --libs check

Tobias Droste tdroste at gmx.de
Thu Oct 5 17:29:02 UTC 2017


Reviewed-by: Tobias Droste <tdroste at gmx.de>

Am Donnerstag, 5. Oktober 2017, 12:19:03 CEST schrieb Emil Velikov:
> From: Emil Velikov <emil.velikov at collabora.com>
> 
> Prior to the refactor/cleanup by Tobias one could add an invalid
> component to LLVM_COMPONENTS.
> 
> Since that's no longer the case we can drop the current check.
> 
> Cc: Tobias Droste <tdroste at gmx.de>
> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
> ---
>  configure.ac | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 603256cf812..dd095fafaad 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -2651,9 +2651,6 @@ if test "x$enable_llvm" = xyes; then
>      dnl this was causing the same libraries to be appear multiple times
>      dnl in LLVM_LIBS.
> 
> -    if ! $LLVM_CONFIG --libs ${LLVM_COMPONENTS} >/dev/null; then
> -       AC_MSG_ERROR([Calling ${LLVM_CONFIG} failed])
> -    fi
>      LLVM_LIBS="`$LLVM_CONFIG --libs ${LLVM_COMPONENTS}`"
> 
>      if test "x$enable_llvm_shared_libs" = xyes; then


More information about the mesa-dev mailing list