[Mesa-dev] [PATCH] configure: Trust LLVM >= 4.0 llvm-config --libs for shared libraries
emil.l.velikov at gmail.com
Thu Aug 10 16:45:46 UTC 2017
On 9 August 2017 at 02:00, Michel Dänzer <michel at daenzer.net> wrote:
> On 09/08/17 03:24 AM, Emil Velikov wrote:
>> On 8 August 2017 at 08:23, Michel Dänzer <michel at daenzer.net> wrote:
>>> From: Michel Dänzer <michel.daenzer at amd.com>
>>> No need to manually look for the library files anymore with current
>>> LLVM. This sidesteps the manual method failing when LLVM was built with
>> IIRC you recently spotted that the Suse packaging was building LLVM
>> with separate shared libraries.
> Apparently that's getting fixed though.
>> This is something which gets flagged with this (admittedly) quirky test.
> What do you mean by that?
> I just double-checked that Mesa with this patch still builds fine
> against LLVM built with BUILD_SHARED_LIBS=ON, just like without it.
Right, I miss-remembered what the else block does.
Thanks for the correction.
Topic for another thread:
There's been a handful of issues reported with BUILD_SHARED_LIBS=ON.
Should we consider that unsupported setup and error out?
>> I think we'd want to keep the best of both - just not sure how to
>> exactly do that.
>> --libs/--libfiles was completely busted with LLVM 3.9 and is back to
>> normal with 4.0.
>> Could we use that somehow?
> This patch relies on llvm-config --libs with LLVM >= 4.0. If you mean
> something else, please be more specific.
That's the thing - in some versions --libs is busted while for others
There isn't a thing we could reliably check across the board, so I was
hoping you know of something.
That said, patch does exactly what it says on the
Reviewed-by: Emil Velikov <emil.velikov at collabora.com>
More information about the mesa-dev