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

[Michael Kruse via Phabricator via llvm-commits]

Meinersbur added inline comments.


================
Comment at: tools/bugpoint/CMakeLists.txt:40
 
-if(WITH_POLLY AND LINK_POLLY_INTO_TOOLS)
+if(WITH_POLLY AND LINK_POLLY_INTO_TOOLS AND NOT LLVM_LINK_LLVM_DYLIB)
   target_link_libraries(bugpoint PRIVATE Polly)
----------------
Thanks for taking care of this. Compiling the different ways Polly can be configured (as loadable module/part of LLVM; static/BUILD_SHARED_LIBS/DYLIB; inside the LLVM build tree/standalone; llvm-config/CMake module) is surprisingly hard and we probably did not test all combinations. 


Repository:
  rL LLVM

https://reviews.llvm.org/D51984