[PATCH] D148751: [CMake] Add llvm-lib to Clang bootstrap dependency for LTO builds on Windows

Martin Storsjö via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 19 21:23:43 PDT 2023


mstorsjo added inline comments.


================
Comment at: clang/CMakeLists.txt:616
         -DDYLD_LIBRARY_PATH=${LLVM_LIBRARY_OUTPUT_INTDIR})
-    elseif(NOT WIN32)
+    elseif(WIN32)
+      add_dependencies(clang-bootstrap-deps llvm-lib)
----------------
I think I’d prefer to have the condition be `MSVC`, not `WIN32`, as we don’t want to use `llvm-lib` in the case of mingw.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148751



More information about the cfe-commits mailing list