DiamondLovesYou added a comment. In https://reviews.llvm.org/D51986#1232440, @beanz wrote:
> After line 18 in this file you could do something like: > > if(WITH_POLLY) > list(APPEND LLVM_LINK_COMPONENTS Polly) > endif() > > > Then you can get rid of the `target_link_libraries` call. It turns out this can't be done. Consider the `LLVMLTO` component and its loadable form, `LTO`. Contrast this with Polly: `Polly` is the component and `LLVMPolly` is the loadable module. This naming convention disagreement prevents Polly's use as a component while also not breaking LTO, I've found (Chris, is this what you were referring to?). Specifically, CMake fails because executables can't link to a `MODULE_LIBRARY` target when trying to link `LLVMPolly`. I think the Polly naming scheme should be changed to match LTO's scheme, rather than vice versa, but not my call. Repository: rC Clang https://reviews.llvm.org/D51986 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
