[Lldb-commits] [lldb] b89131c - [lldb] removed no longer needed CMakeDependentOption

Konrad Kleine via lldb-commits lldb-commits at lists.llvm.org
Fri Feb 7 00:23:47 PST 2020


Author: Konrad Kleine
Date: 2020-02-07T09:23:35+01:00
New Revision: b89131cdda5871731a9139664aef2b70c6d72bbd

URL: https://github.com/llvm/llvm-project/commit/b89131cdda5871731a9139664aef2b70c6d72bbd
DIFF: https://github.com/llvm/llvm-project/commit/b89131cdda5871731a9139664aef2b70c6d72bbd.diff

LOG: [lldb] removed no longer needed CMakeDependentOption

Summary:
In D66791 I've introduced this [[ https://cmake.org/cmake/help/latest/module/CMakeDependentOption.html | `CMakeDependentOption` ]] but in
D71306 @JDevlieghere has changed the way optional dependencies
are handled in LLDB. Today there's no occurence of
`cmake_dependent_option` inside the lldb source tree.

That's why this include can be removed.

Reviewers: JDevlieghere, labath

Reviewed By: labath

Subscribers: labath, mgorny, lldb-commits, JDevlieghere

Tags: #lldb

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

Added: 
    

Modified: 
    lldb/cmake/modules/LLDBConfig.cmake

Removed: 
    


################################################################################
diff  --git a/lldb/cmake/modules/LLDBConfig.cmake b/lldb/cmake/modules/LLDBConfig.cmake
index 4a15a343ee1d..fb4512a87998 100644
--- a/lldb/cmake/modules/LLDBConfig.cmake
+++ b/lldb/cmake/modules/LLDBConfig.cmake
@@ -1,6 +1,5 @@
 include(CheckCXXSymbolExists)
 include(CheckTypeSize)
-include(CMakeDependentOption)
 
 set(LLDB_PROJECT_ROOT ${CMAKE_CURRENT_SOURCE_DIR})
 set(LLDB_SOURCE_ROOT "${CMAKE_CURRENT_SOURCE_DIR}/source")


        


More information about the lldb-commits mailing list