[LLVMdev] proposal to avoid zlib dependency.

[Mueller-Roemer, Johannes Sebastian]

Dylib/so/shared doesn't work on windows yet. 
________________________________________
From: Filip Pizlo [fpizlo at apple.com]
Sent: Thursday, September 18, 2014 5:23 PM
To: Chandler Carruth
Cc: Mueller-Roemer, Johannes Sebastian; LLVM Developers Mailing List
Subject: Re: [LLVMdev] proposal to avoid zlib dependency.

Once LLVM is built and used as a dylib, we'd just mark the zlib symbols as hidden. :-)

I tend to think that bundling is ugly because it makes it harder for a distributor or integrator to pick the exact versions of all of the components in the system. I don't know if this was mentioned already, but maybe we can have a compromise where a configure option can be used to select between the system zlib and whatever LLVM bundles. The default can be system zlib with an automatic fallback of using the bundled zlib (or zlib equivelant).

-Filip

On Sep 18, 2014, at 3:03 AM, Chandler Carruth <chandlerc at google.com<mailto:chandlerc at google.com>> wrote:


On Thu, Sep 18, 2014 at 2:58 AM, Mueller-Roemer, Johannes Sebastian <Johannes.Sebastian.Mueller-Roemer at igd.fraunhofer.de<mailto:Johannes.Sebastian.Mueller-Roemer at igd.fraunhofer.de>> wrote:
There is still one reason this should NOT be done:

If some other library which uses LLVM wants to use zlib (either the system version or one built by hand) we will have linker issues with multiple definition, as LLVM only works with static libraries on Windows. Unless LLVM uses a custom prefix for its internal ZLib, which would “only” lead to more binary bloat.

If we do this at all, we would clearly have to use an LLVM-specific name for any functions / routines / etc.

While this might be extraneous space in the binary, compared to even the smallest part of LLVM, I suspect it would be lost in the noise.
_______________________________________________
LLVM Developers mailing list
LLVMdev at cs.uiuc.edu<mailto:LLVMdev at cs.uiuc.edu>         http://llvm.cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev