[PATCH] D50668: unittests: Don't install TestPlugin.so

[Philip Pfaffe via Phabricator via llvm-commits]

philip.pfaffe added a comment.

This makes complete sense to me, but I'm wondering whether add_llvm_loadable_module is the right macro to use here in the first place. All it does is call add_llvm_library with a MODULE argument, and set up the install.

So it seems to me that add_llvm_library(TestPlugin MODULE BUILDTREE_ONLY ...) should work just as well, without changing the setup. Does that make sense?


Repository:
  rL LLVM

https://reviews.llvm.org/D50668