[PATCH] D85627: [polly][cmake] Don't build LLVMPolly.so without PIC

Mon Aug 10 06:32:20 PDT 2020

Meinersbur added a comment.

Could you outline your intended solution? If I read correctly, with `LLVM_ENABLE_PIC=OFF`, no Polly module will be built, making Polly useless unless Polly is built in-tree and `LLVM_POLLY_LINK_INTO_TOOLS` is used.

================
Comment at: polly/lib/CMakeLists.txt:141
+if (MSVC OR NOT LLVM_ENABLE_PIC)
   # Add dummy target, because loadable modules are not supported on Windows
   add_custom_target(LLVMPolly)
----------------
Please update the comment

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D85627/new/

https://reviews.llvm.org/D85627