[PATCH] D29146: Link LLVM dynamically on Windows, deploy Universal CRT dlls.

Vadim Chugunov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 26 13:41:12 PST 2017


vadimcn added a comment.

> Could we make CMAKE_INSTALL_UCRT_LIBRARIES the default when building for Windows? Would there be any downside to that?

I don't think it's necessary.  People building locally will already UCRT installed since they have MSVC.  It's only the "official" builds that might get installed on a bare machine that might need those dlls.
Besides, CMake will mark WINDOWS_KITS_REDIST_DIR path with an error if builder's machine doesn't have a recent version of Windows 10 SDK installed (UCRT was not redistributable until late 2015).  This might be confusing.


Repository:
  rL LLVM

https://reviews.llvm.org/D29146





More information about the llvm-commits mailing list