[llvm-dev] Building SVN head with CMake - shared libraries?

[Chris Bieneman via llvm-dev]

> On Jan 14, 2016, at 11:43 AM, Ismail Donmez <ismail at i10z.com> wrote:
> 
> On Thu, Jan 14, 2016 at 9:31 PM, Chris Bieneman via llvm-dev
> <llvm-dev at lists.llvm.org <mailto:llvm-dev at lists.llvm.org>> wrote:
>> 
>> On Jan 14, 2016, at 11:22 AM, Mehdi Amini <mehdi.amini at apple.com> wrote:
>> 
>> 
>> On Jan 14, 2016, at 9:38 AM, Chris Bieneman via llvm-dev
>> <llvm-dev at lists.llvm.org> wrote:
>> 
>> 
>> On Jan 14, 2016, at 5:18 AM, Dan Liew <dan at su-root.co.uk> wrote:
>> 
>> On 14 January 2016 at 11:24, David Jones via llvm-dev
>> <llvm-dev at lists.llvm.org> wrote:
>> 
>> Thanks - I'll try this tonight.
>> 
>> Assuming it works, should these variables be added to the docs at
>> http://llvm.org/docs/CMake.html ?
>> 
>> 
>> Yes.
>> 
>> 
>> It probably makes sense to add LLVM_BUILD_LLVM_DYLIB, but I would discourage
>> adding BUILD_SHARED_LIBS to that document. BUILD_SHARED_LIBS is rarely what
>> people actually want, and a lot of people gravitate to it because the
>> wording is similar to the autoconf —enable-shared flag.
>> 
>> 
>> Considering that BUILD_SHARED_LIBS is rarely what people want, wouldn’t it
>> make sense to document it to explicitly discourage people from using it and
>> avoid them surprising behavior?
>> 
>> 
>> Honestly, I’m not sure *anyone* actually wants that behavior, and I’d kinda
>> like to remove the option entirely.
> 
> Do I understand it correctly that LLVM_BUILD_LLVM_DYLIB produces a one
> big libLLVM.so and a lot of static libraries? Btw currently for
> openSUSE builds we do utilize BUILD_SHARED_LIBS so please don't remove
> it.

I’m kinda scared that you’re using it. What are you trying to accomplish that you are using it? Generally having LLVM split among that many dynamically loaded libraries results in significant performance regressions.

-Chris

> 
> Thanks!

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20160114/ac500738/attachment.html>