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

[Tom Stellard]

On 10/05/2017 06:33 AM, Michel Dänzer 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.
> 
> 
>> 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 agree with this, symbol versioning should solve the issues with  symbol
collisions, so why change this now?

Also, do we have a linker script or some other mechanism to prevent
mesa's libraries from exporting LLVM symbols when statically linked?

-Tom