[PATCH] D72372: [CMake] Force static linking for registered plugins on Windows.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 8 16:05:47 PST 2020


MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.

I don't really understand the mechanism on Windows. Hope @serge-sans-paille	can confirm.



================
Comment at: llvm/cmake/modules/AddLLVM.cmake:858
 
-  option(LLVM_${name_upper}_LINK_INTO_TOOLS "Statically link ${name} into tools (if available)" OFF)
+  if (WIN32)
+    # Dynamically linked pass plugins are not supported on Windows: the plugin
----------------
The convention seems that there is no space before `(`


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

https://reviews.llvm.org/D72372





More information about the llvm-commits mailing list