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

[⚛]

On Wed, Jul 27, 2016 at 2:44 PM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> 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

After seeing your output of llvm-config --system-libs, it turns out
that it is a Gentoo-specific issue when using static libs
(https://archives.gentoo.org/gentoo-commits/message/0f2dc0024bf1d53756de988602d086da).

I will fix my llvm.ebuild locally and cancel this patch if the fix works.