[Mesa-dev] [PATCH] configure: additional libs for LLVM static linking

[Emil Velikov]

On 27 July 2016 at 12:36, Jan Ziak <0xe2.0x9a.0x9b at gmail.com> wrote:
> Signed-off-by: Jan Ziak (http://atom-symbol.net) <0xe2.0x9a.0x9b at gmail.com>
> ---
>  configure.ac | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index 5c196a9..58c2db4 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -2194,6 +2194,7 @@ if test "x$enable_gallium_llvm" = xyes; then
>
>              LLVM_COMPONENTS="${LLVM_COMPONENTS} all-targets ipo linker instrumentation"
>              LLVM_COMPONENTS="${LLVM_COMPONENTS} irreader option objcarcopts profiledata"
> +            LLVM_COMPONENTS="${LLVM_COMPONENTS} coverage"
It's quite strange that nobody has noticed this yet. Is this something
new with llvm 4.0svn, misuse on mesa side, other ?

>          fi
>          DEFINES="${DEFINES} -DHAVE_LLVM=0x0$LLVM_VERSION_INT -DMESA_LLVM_VERSION_PATCH=$LLVM_VERSION_PATCH"
>          MESA_LLVM=1
> @@ -2536,7 +2536,7 @@ if test "x$MESA_LLVM" != x0; then
>          dnl We need to link to llvm system libs when using static libs
>          dnl However, only llvm 3.5+ provides --system-libs
>          if test $LLVM_VERSION_MAJOR -eq 3 -a $LLVM_VERSION_MINOR -ge 5; then
> -            LLVM_LIBS="$LLVM_LIBS `$LLVM_CONFIG --system-libs`"
> +            LLVM_LIBS="$LLVM_LIBS `$LLVM_CONFIG --system-libs` $(pkg-config --libs ncurses zlib)"
On my 3.8.0 setup this produces the following... which looks a bit odd.
-lrt -ldl -lcurses -lpthread -lz -lm -lncursesw -lz

Please add a small summary on "why/how" in the commit message for the changes.

Thanks
Emil