[Mesa-dev] [PATCH] configure: Check llvm-config --shared-mode

Andy Furniss adf.lists at gmail.com
Tue Aug 15 09:47:58 UTC 2017


Thanks, I am also building OK with this + my posted llvm config which
maybe should be optimised somehow rather than arrived at by trial and 
error :-)

Michel Dänzer wrote:
> From: Michel Dänzer <michel.daenzer at amd.com>
> 
> https://bugs.llvm.org/show_bug.cgi?id=6823 still affects current LLVM.
> llvm-config --libs only reports the single shared library if LLVM was
> built with -DLLVM_LINK_LLVM_DYLIB=ON. llvm-config --shared-mode reports
> "shared" in that case, "static" otherwise (even if LLVM was built with
> -DLLVM_BUILD_LLVM_DYLIB=ON).
> 
> Fixes: 3d8da1f678e1 ("configure: Trust LLVM >= 4.0 llvm-config --libs
>                        for shared libraries")
> Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>
> ---
>   configure.ac | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 3fe47c7bbc..de15ac2fca 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -2624,7 +2624,7 @@ if test "x$enable_llvm" = xyes; then
>       LLVM_LIBS="`$LLVM_CONFIG --libs ${LLVM_COMPONENTS}`"
>   
>       if test "x$enable_llvm_shared_libs" = xyes; then
> -        if test $LLVM_VERSION_MAJOR -lt 4; then
> +        if test `$LLVM_CONFIG --shared-mode` = static; then
>               dnl llvm-config may not give the right answer when llvm is a built as a
>               dnl single shared library, so we must work the library name out for
>               dnl ourselves.
> 



More information about the mesa-dev mailing list