[Mesa-dev] [PATCH] configure.ac: handle llvm built with cmake in one shared library

Emil Velikov emil.l.velikov at gmail.com
Wed Sep 30 03:47:39 PDT 2015


On 29 September 2015 at 23:53, Dave Airlie <airlied at gmail.com> wrote:
> On 30 September 2015 at 01:34, Laurent Carlier <lordheavym at gmail.com> wrote:
>> llvm can be built with cmake in a libray with the name libLLVM.so.$version
>> Tested with both llvm-3.7.0 and llvm-3.8.0svn
>>
>> v2: check and use llvm build-system feature, update comments
>>
>
> just to reiterate, this should be fixed in llvm.
>
> cmake and autoconf should produce libLLVM$version.so since their ABI
> isn't compatible across releases.
>
Interestingly, there seems to be more interest in adding 'hacks' [1]
in llvm+users over fixing the llvm cmake issue(s).

Sorry Tom, but I feel that adding --build-system, so that users can
workaround the bugs is not the most productive approach. Can we please
fix this before the autoconf build is removed? Btw what is happening
with the no-rtti bug - last time I've looked, there was some 2+yo
patch(es) that never got merged.

If the llvm folk isn't sure how to fix this, you can check waffle's approach[2].

Thanks
Emil

[1] https://github.com/llvm-mirror/llvm/commit/800a5b06646060e4a5622a79153336728112ca3b
[2] https://github.com/waffle-gl/waffle/blob/master/cmake/Modules/WaffleDefineVersion.cmake


More information about the mesa-dev mailing list