[PATCH] D75068: libclang: Add static build support for Windows

Mon Apr 27 03:43:01 PDT 2020

cristian.adam added a comment.

In D75068#2004072 <https://reviews.llvm.org/D75068#2004072>, @davezarzycki wrote:

> Rather than wholly reverting this, I committed a fix: 665471907a5c072c6653a38c35f35e5d54cef220 <https://reviews.llvm.org/rG665471907a5c072c6653a38c35f35e5d54cef220>
>
> The problem is that "NOT LIBCLANG_BUILD_STATIC" does not imply PIC, so there is no point in trying to build the SHARED version. Please let me know if I somehow missed something. Thanks!

@thakis  has a use case at https://reviews.llvm.org/D74907 building a SHARED libclang with `LLVM_ENABLE_PIC` set to `OFF`.

Would the variant `if(LLVM_ENABLE_PIC OR (WIN32 AND NOT LIBCLANG_BUILD_STATIC))`  work for the Fedora case?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75068