[Mesa-dev] [PATCH 7/7] configure.ac: static link LLVM by default

[Emil Velikov]

On 5 October 2017 at 14:33, Michel Dänzer <michel at daenzer.net> wrote:
> On 05/10/17 12:19 PM, Emil Velikov wrote:
>> From: Emil Velikov <emil.velikov at collabora.com>
>>
>> A while back Michel reported that LLVM has symbol versioning to avoid
>> symbol collisions. Based on observations LLVM 5.0 is the first upstream
>> version to actually has it.
>
> Not exactly. Adam Jackson originally added symbol versioning in LLVM 3.6
> (in SVN r214418), but it was only effective when LLVM was built with
> autotools. As of 5.0, it's effective with cmake as well.
>
>
Hmm that would explain it - the builds I've checked were using cmake :-(
Any objections if I add that quote to the commit message?

>> Since symbol collisions do come up again and again (fortunately not so
>> often) let's flip the switch back to static.
>
> It seems a bit weird to make this change now, that LLVM is solving the
> issue for good. But I don't feel strongly about it.
>
I should have mentioned:
Quick search suggests that LLVM 5.0 is available on:
 - Debian (testing/unstable)
 - Ubuntu (artful)
 - Arch
 - Gentoo

Of which only Arch has it as "default". Even Fedora 27 (due in a
couple of months) is shipping 4.0, seemingly.
The future is bright, but we're not there yet.

Thinking about it - I could add that as well to the commit message.

-Emil