[PATCH] D51984: Fixes for `LLVM_LINK_LLVM_DYLIB` && Polly.

[Michael Kruse via Phabricator via llvm-commits]

Meinersbur added a subscriber: grosser.
Meinersbur added a comment.

In https://reviews.llvm.org/D51984#1233719, @beanz wrote:

> +      # LLVMPolly is the Polly loadable module target, the static archive is just Polly
>  It is unfortunate that Polly doesn't match the naming conventions of other LLVM components, but we have a lot of special case handling for this kind of thing anyways.


This is historic: At the beginning there was just "LLVMPolly", the loadable module (probably to match the convention). When linking as a static component of LLVM, "LLVMPolly" was taken, so it was called just "Polly".
If @grosser agrees, we can switch the naming (with the problem that the command line `-load LLVMPolly.so` won't work anymore).


Repository:
  rL LLVM

https://reviews.llvm.org/D51984