[PATCH] D32392: [Polly][CMake] Allow Polly to transitively pull in its dependencies into host programs.

[Tobias Grosser via Phabricator via llvm-commits]

grosser accepted this revision.
grosser added a comment.
This revision is now accepted and ready to land.

Hi Michael,

this seems to be a nice cmake cleanup now finally adding Polly with the right native cmake tools. I think it is OK to have Polly as part of libLLVM.so, as this is only the case if Polly is actually checked out. I also think it is OK to expose the core library not to external users. It is hard to guess which build artifacts they might need, if we neither know about their existence nor about their use cases. I personally would guess there are none, but if there are I am sure they will let us know when we break them. In this case we can always try to address their use case.

Best,
Tobias


https://reviews.llvm.org/D32392