Revert r122395, "Fixes file extension for loadable modules on OS X."
authorNAKAMURA Takumi <geek4civic@gmail.com>
Tue, 4 Feb 2014 13:33:13 +0000 (13:33 +0000)
committerNAKAMURA Takumi <geek4civic@gmail.com>
Tue, 4 Feb 2014 13:33:13 +0000 (13:33 +0000)
In trunk, every users assume add_llvm_loadable_module as "loadable module" and no one sets neither SHARED, ... nor also MODULE!
Unfortunately, all loadable modules were linked as not "MODULE" but "SHARED".

If this change caused any regressions, I wish guys to fix it properly. ;)

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@200762 91177308-0d34-0410-b5e6-96231b3b80d8

cmake/modules/AddLLVM.cmake

index 0c9abcad5090fb670757bef65fd6c5d39d2d5297..f1da76d8d319d236dd1925e7c94cb61bdb0d6e2a 100644 (file)
@@ -217,14 +217,7 @@ ${name} ignored.")
     # Add empty "phony" target
     add_custom_target(${name})
   else()
-    llvm_process_sources( ALL_FILES ${ARGN} )
-    if (MODULE)
-      set(libkind MODULE)
-    else()
-      set(libkind SHARED)
-    endif()
-
-    add_library( ${name} ${libkind} ${ALL_FILES} )
+    add_library(${name} MODULE ${ALL_FILES})
     set_output_directory(${name} ${LLVM_RUNTIME_OUTPUT_INTDIR} ${LLVM_LIBRARY_OUTPUT_INTDIR})
     set_target_properties( ${name} PROPERTIES PREFIX "" )
     llvm_update_compile_flags(${name})