[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