[polly][cmake] patch to build monolithic shared library

Tobias Grosser tobias at grosser.es
Wed Feb 26 15:17:52 PST 2014


On 02/27/2014 12:07 AM, David Fang wrote:
> Hi,
>      I've recently refreshed my patch to link polly as a single shared
> lib.  This was in response to the original report at:
> http://llvm.org/bugs/show_bug.cgi?id=14833
>
> Live patch:
> https://github.com/fangism/polly/compare/master...shared-lib-patch.diff
>
> I don't really know whether a single shared-lib is preferred over
> sub-libraries.  This was intended as a workaround, but if parties are
> interested, this patch could be cleaned up.

Hi Andreas,

I remember we moved Polly to multiple libraries mostly due to the needs 
in your polly-prof project. I remember from our recent discussions that 
this is now in fact not needed anymore, but that it is sufficient to 
have one large Polly library and one that contains the core 
functionality of RegisterPasses.cpp.

As Polly is in fact not that large and more importantly the multi-module 
built is broken anyway, I wonder if it may make sense to just follow 
David's patch and go for a single module? I remember you also have some 
patches on top of the Polly cmake files, so maybe we can find a solution 
that allows both of you to drop your patches. What do you think?

Cheers,
Tobias





More information about the llvm-commits mailing list