[PATCH] D51984: Fixes for `LLVM_LINK_LLVM_DYLIB` && Polly.
Michael Kruse via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 13 14:04:53 PDT 2018
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
More information about the llvm-commits
mailing list