[PATCH] D70026: [cmake] Always build the libLLVM shared library

Chris Bieneman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 13 11:20:21 PST 2019


beanz added inline comments.


================
Comment at: llvm/CMakeLists.txt:573-577
+  set(LLVM_BUILD_LLVM_DYLIB OFF)
 else()
   option(LLVM_LINK_LLVM_DYLIB "Link tools against the libllvm dynamic library" OFF)
   option(LLVM_BUILD_LLVM_C_DYLIB "Build libllvm-c re-export library (Darwin only)" OFF)
+  set(LLVM_BUILD_LLVM_DYLIB ON)
----------------
tstellar wrote:
> smeenai wrote:
> > tstellar wrote:
> > > @beanz Does setting the variables here override the cache variables?
> > Yup, setting the variable here will override the cached value. Any reason to prefer this to just removing all uses of the variable though?
> I thought it would be cleaner to keep the variable than to replace it with if (MSVC) everywhere.  
There is really only one place the `if (MSVC)` check would be needed, in the llvm-shlib/CMakeLists.txt file. All the other uses of the variable could disappear.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70026





More information about the llvm-commits mailing list