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

serge via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 8 22:04:01 PST 2020


serge-sans-paille added a comment.

I like the idea but

1. I'm unsure it's fine to have a different default for different platform (but that's arguable)
2. This would link the *example* Bye plugin statically, which is not something we want (and that's critical).

One workaround would be to warn the user if the setting is set to OFF on Windows, or disable the test on Windows?


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

https://reviews.llvm.org/D72372





More information about the llvm-commits mailing list