From e538c6fc3048e1fb1a58da879275d6804186856e Mon Sep 17 00:00:00 2001 From: walter erquinigo Date: Mon, 3 Apr 2023 11:32:53 -0500 Subject: [PATCH] [LLDB] Ensure LLDB symbols are exported in LLDB_EXPORT_ALL_SYMBOLS is provided. If we want to export all lldb symbols (i.e LLDB_EXPORT_ALL_SYMBOLS=ON), we need to use default visibility for all LLDB libraries even if a global CMAKE_CXX_VISIBILITY_PRESET=hidden is present. In fact, there are cases in which a global llvm configuration wants CMAKE_CXX_VISIBILITY_PRESET as hidden but simultaneously LLDB_EXPORT_ALL_SYMBOLS=ON is also needed to be able to develop out-of-tree lldb plugins. Differential Revision: https://reviews.llvm.org/D147453 --- lldb/cmake/modules/AddLLDB.cmake | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/lldb/cmake/modules/AddLLDB.cmake b/lldb/cmake/modules/AddLLDB.cmake index f2d96df..d47a30f 100644 --- a/lldb/cmake/modules/AddLLDB.cmake +++ b/lldb/cmake/modules/AddLLDB.cmake @@ -165,6 +165,13 @@ function(add_lldb_library name) else() set_target_properties(${name} PROPERTIES FOLDER "lldb libraries") endif() + + # If we want to export all lldb symbols (i.e LLDB_EXPORT_ALL_SYMBOLS=ON), we + # need to use default visibility for all LLDB libraries even if a global + # `CMAKE_CXX_VISIBILITY_PRESET=hidden`is present. + if (LLDB_EXPORT_ALL_SYMBOLS) + set_target_properties(${name} PROPERTIES CXX_VISIBILITY_PRESET default) + endif() endfunction(add_lldb_library) function(add_lldb_executable name) -- 2.7.4