[Lldb-commits] [lldb] r285894 - Link lldb-mi only to the llvm components it uses

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Thu Nov 3 03:52:18 PDT 2016


Author: labath
Date: Thu Nov  3 05:52:17 2016
New Revision: 285894

URL: http://llvm.org/viewvc/llvm-project?rev=285894&view=rev
Log:
Link lldb-mi only to the llvm components it uses

Summary:
liblldb does not re-export the llvm library contained within, so lldb-mi needs to
manage its own dependencies. Right now it only uses the llvm support library.

Reviewers: beanz, zturner, tfiala, clayborg, abidh

Subscribers: ki.stfu, mgorny, lldb-commits

Differential Revision: https://reviews.llvm.org/D26190

Modified:
    lldb/trunk/tools/lldb-mi/CMakeLists.txt

Modified: lldb/trunk/tools/lldb-mi/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/tools/lldb-mi/CMakeLists.txt?rev=285894&r1=285893&r2=285894&view=diff
==============================================================================
--- lldb/trunk/tools/lldb-mi/CMakeLists.txt (original)
+++ lldb/trunk/tools/lldb-mi/CMakeLists.txt Thu Nov  3 05:52:17 2016
@@ -1,5 +1,3 @@
-include(${LLDB_PROJECT_ROOT}/cmake/LLDBDependencies.cmake)
-
 set(LLDB_MI_SOURCES
   MICmdArgContext.cpp
   MICmdArgSet.cpp
@@ -83,6 +81,9 @@ if ( CMAKE_SYSTEM_NAME MATCHES "Windows"
     )
 endif ()
 
+# We need to include the llvm components we depend on manually, as liblldb does
+# not re-export those.
+set(LLVM_LINK_COMPONENTS Support)
 add_lldb_executable(lldb-mi ${LLDB_MI_SOURCES})
 
 target_link_libraries(lldb-mi liblldb)
@@ -90,8 +91,6 @@ if ( NOT CMAKE_SYSTEM_NAME MATCHES "Wind
   target_link_libraries(lldb-mi pthread)
 endif ()
 
-llvm_config(lldb-mi ${LLVM_LINK_COMPONENTS})
-
 set_target_properties(lldb-mi PROPERTIES VERSION ${LLDB_VERSION})
 
 install(TARGETS lldb-mi




More information about the lldb-commits mailing list