[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