[PATCH] D51748: cmake: Refactor add_llvm_loadable_module()

[Chris Bieneman via Phabricator via llvm-commits]

beanz added inline comments.


================
Comment at: cmake/modules/AddLLVM.cmake:677
+  add_llvm_library(${name} MODULE ${ARGN})
   if(NOT TARGET ${name})
     # Add empty "phony" target
----------------
I don't believe there is any situation where calling add_llvm_library with `MODULE` would ever result in not creating the target, so this condition is probably not needed, which makes me think we should probably just get rid of `add_llvm_loadable_module` entirely.


Repository:
  rL LLVM

https://reviews.llvm.org/D51748