From 7a6252158e2fa1f3dfd0a4c846db0c6365d3f3ea Mon Sep 17 00:00:00 2001 From: Pavel Labath Date: Mon, 5 Dec 2016 11:15:36 +0000 Subject: [PATCH] Clean up some use of __ANDROID_NDK__ in the cmake files Rationale: scripts/Python/modules: android is excluded at a higher level, so no point in checking here tools/lldb-mi: lldb-mi builds fine (with some cosmetic tweaks) on android, and there is no reason it shouldn't. tools/lldb-server: LLDB_DISABLE_LIBEDIT/CURSES already take the platform into account, so there is no point in checking again. I am reasonably confident this should not break the build on any platform, but I'll keep an eye out on the bots. llvm-svn: 288661 --- lldb/scripts/Python/modules/CMakeLists.txt | 2 +- lldb/tools/CMakeLists.txt | 8 +++----- lldb/tools/lldb-mi/CMakeLists.txt | 2 +- lldb/tools/lldb-mi/MIUtilString.cpp | 1 + lldb/tools/lldb-server/CMakeLists.txt | 16 +++++++--------- 5 files changed, 13 insertions(+), 16 deletions(-) diff --git a/lldb/scripts/Python/modules/CMakeLists.txt b/lldb/scripts/Python/modules/CMakeLists.txt index 396d447ff267..01ab5771bda0 100644 --- a/lldb/scripts/Python/modules/CMakeLists.txt +++ b/lldb/scripts/Python/modules/CMakeLists.txt @@ -6,6 +6,6 @@ if (CXX_SUPPORTS_NO_MACRO_REDEFINED) endif () # build the Python readline suppression module only on Linux -if (CMAKE_SYSTEM_NAME MATCHES "Linux" AND NOT __ANDROID_NDK__) +if (CMAKE_SYSTEM_NAME MATCHES "Linux") add_subdirectory(readline) endif() diff --git a/lldb/tools/CMakeLists.txt b/lldb/tools/CMakeLists.txt index 49af928c3815..86b9621a198e 100644 --- a/lldb/tools/CMakeLists.txt +++ b/lldb/tools/CMakeLists.txt @@ -2,11 +2,9 @@ if (CMAKE_SYSTEM_NAME MATCHES "Darwin") add_subdirectory(darwin-debug) add_subdirectory(debugserver) endif() - add_subdirectory(argdumper) - add_subdirectory(driver) -if (NOT __ANDROID_NDK__) - add_subdirectory(lldb-mi) -endif() +add_subdirectory(argdumper) +add_subdirectory(driver) +add_subdirectory(lldb-mi) if (LLDB_CAN_USE_LLDB_SERVER) add_subdirectory(lldb-server) endif() diff --git a/lldb/tools/lldb-mi/CMakeLists.txt b/lldb/tools/lldb-mi/CMakeLists.txt index f42ec02bedae..1e73b8200a82 100644 --- a/lldb/tools/lldb-mi/CMakeLists.txt +++ b/lldb/tools/lldb-mi/CMakeLists.txt @@ -87,7 +87,7 @@ set(LLVM_LINK_COMPONENTS Support) add_lldb_executable(lldb-mi ${LLDB_MI_SOURCES}) target_link_libraries(lldb-mi liblldb) -if ( NOT CMAKE_SYSTEM_NAME MATCHES "Windows" ) +if (HAVE_LIBPTHREAD) target_link_libraries(lldb-mi pthread) endif () diff --git a/lldb/tools/lldb-mi/MIUtilString.cpp b/lldb/tools/lldb-mi/MIUtilString.cpp index 2ea17fe0840b..3e6911f68189 100644 --- a/lldb/tools/lldb-mi/MIUtilString.cpp +++ b/lldb/tools/lldb-mi/MIUtilString.cpp @@ -8,6 +8,7 @@ //===----------------------------------------------------------------------===// // Third party headers +#include #include // for PRIx8 #include // for ULONG_MAX #include // std::unique_ptr diff --git a/lldb/tools/lldb-server/CMakeLists.txt b/lldb/tools/lldb-server/CMakeLists.txt index 990c4195eda5..bc238282ea78 100644 --- a/lldb/tools/lldb-server/CMakeLists.txt +++ b/lldb/tools/lldb-server/CMakeLists.txt @@ -108,15 +108,13 @@ set( CLANG_USED_LIBS ) set(LLDB_SYSTEM_LIBS) -if (NOT CMAKE_SYSTEM_NAME MATCHES "Windows" AND NOT __ANDROID_NDK__) - if (NOT LLDB_DISABLE_LIBEDIT) - list(APPEND LLDB_SYSTEM_LIBS edit) - endif() - if (NOT LLDB_DISABLE_CURSES) - list(APPEND LLDB_SYSTEM_LIBS ${CURSES_LIBRARIES}) - if(LLVM_ENABLE_TERMINFO AND HAVE_TERMINFO) - list(APPEND LLDB_SYSTEM_LIBS ${TERMINFO_LIBS}) - endif() +if (NOT LLDB_DISABLE_LIBEDIT) + list(APPEND LLDB_SYSTEM_LIBS edit) +endif() +if (NOT LLDB_DISABLE_CURSES) + list(APPEND LLDB_SYSTEM_LIBS ${CURSES_LIBRARIES}) + if(LLVM_ENABLE_TERMINFO AND HAVE_TERMINFO) + list(APPEND LLDB_SYSTEM_LIBS ${TERMINFO_LIBS}) endif() endif()