[Lldb-commits] [PATCH] D156997: [LLDB][CMake][NFC] Remove unused LLDB_LINKER_SUPPORTS_GROUPS variable
J. Ryan Stinnett via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Aug 3 06:03:59 PDT 2023
jryans created this revision.
jryans added a reviewer: JDevlieghere.
Herald added a subscriber: ekilmer.
Herald added a project: All.
jryans added a project: LLDB.
jryans published this revision for review.
Herald added a subscriber: lldb-commits.
The `LLDB_LINKER_SUPPORTS_GROUPS` CMake variable was added back in 2019 as part
of https://reviews.llvm.org/D71306, but it appears to have been unused even
then. This removes the unused variable.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D156997
Files:
lldb/cmake/modules/LLDBConfig.cmake
Index: lldb/cmake/modules/LLDBConfig.cmake
===================================================================
--- lldb/cmake/modules/LLDBConfig.cmake
+++ lldb/cmake/modules/LLDBConfig.cmake
@@ -17,12 +17,6 @@
"`CMakeFiles'. Please delete them.")
endif()
-set(LLDB_LINKER_SUPPORTS_GROUPS OFF)
-if (LLVM_COMPILER_IS_GCC_COMPATIBLE AND NOT "${CMAKE_SYSTEM_NAME}" MATCHES "Darwin")
- # The Darwin linker doesn't understand --start-group/--end-group.
- set(LLDB_LINKER_SUPPORTS_GROUPS ON)
-endif()
-
macro(add_optional_dependency variable description package found)
cmake_parse_arguments(ARG
"QUIET"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D156997.546823.patch
Type: text/x-patch
Size: 615 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20230803/5ce2f8eb/attachment.bin>
More information about the lldb-commits
mailing list