https://github.com/foxtran updated https://github.com/llvm/llvm-project/pull/128196
>From 4683735bef8f43a5e39be6aedffd76c47e43beb2 Mon Sep 17 00:00:00 2001 From: "Igor S. Gerasimov" <i.s....@ya.ru> Date: Fri, 21 Feb 2025 17:33:25 +0100 Subject: [PATCH 1/2] Use STATUS instead of explicit '--' in MESSAGE --- lldb/source/API/CMakeLists.txt | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lldb/source/API/CMakeLists.txt b/lldb/source/API/CMakeLists.txt index 147b30f3b0026..47b60ef0895a3 100644 --- a/lldb/source/API/CMakeLists.txt +++ b/lldb/source/API/CMakeLists.txt @@ -206,23 +206,23 @@ if (NOT CMAKE_SYSTEM_NAME MATCHES "Windows") # If we're not exporting all symbols, we'll want to explicitly set # the exported symbols here. This prevents 'log enable --stack ...' # from working on some systems but limits the liblldb size. - MESSAGE("-- Symbols (liblldb): exporting all symbols from the lldb namespace") + MESSAGE(STATUS "Symbols (liblldb): exporting all symbols from the lldb namespace") add_llvm_symbol_exports(liblldb ${CMAKE_CURRENT_SOURCE_DIR}/liblldb.exports) elseif (NOT LLDB_EXPORT_ALL_SYMBOLS_EXPORTS_FILE) # Don't use an explicit export. Instead, tell the linker to export all symbols. - MESSAGE("-- Symbols (liblldb): exporting all symbols from the lldb and lldb_private namespaces") + MESSAGE(STATUS "Symbols (liblldb): exporting all symbols from the lldb and lldb_private namespaces") MESSAGE(WARNING "Private LLDB symbols frequently change and no API stability is guaranteed. " "Only the SB API is guaranteed to be stable.") add_llvm_symbol_exports(liblldb ${CMAKE_CURRENT_SOURCE_DIR}/liblldb-private.exports) else () - MESSAGE("-- Symbols (liblldb): exporting all symbols specified in the exports " + MESSAGE(STATUS "Symbols (liblldb): exporting all symbols specified in the exports " " file '${LLDB_EXPORT_ALL_SYMBOLS_EXPORTS_FILE}'") MESSAGE(WARNING "Private LLDB symbols frequently change and no API stability is guaranteed. " "Only the SB API is guaranteed to be stable.") add_llvm_symbol_exports(liblldb "${LLDB_EXPORT_ALL_SYMBOLS_EXPORTS_FILE}") endif() elseif (LLDB_EXPORT_ALL_SYMBOLS) - MESSAGE("-- Symbols (liblldb): exporting all symbols from the lldb and lldb_private namespaces") + MESSAGE(STATUS "Symbols (liblldb): exporting all symbols from the lldb and lldb_private namespaces") # Pull out the various lldb libraries linked into liblldb, these will be used # when looking for symbols to extract. We ignore most plugin libraries here, >From c8405f65bd8a4eeaad1002e5f7feeef41b85f25b Mon Sep 17 00:00:00 2001 From: "Igor S. Gerasimov" <i.s....@ya.ru> Date: Fri, 21 Feb 2025 17:47:21 +0100 Subject: [PATCH 2/2] Use message in lower case --- lldb/source/API/CMakeLists.txt | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/lldb/source/API/CMakeLists.txt b/lldb/source/API/CMakeLists.txt index 47b60ef0895a3..9d6a7d8850286 100644 --- a/lldb/source/API/CMakeLists.txt +++ b/lldb/source/API/CMakeLists.txt @@ -206,23 +206,23 @@ if (NOT CMAKE_SYSTEM_NAME MATCHES "Windows") # If we're not exporting all symbols, we'll want to explicitly set # the exported symbols here. This prevents 'log enable --stack ...' # from working on some systems but limits the liblldb size. - MESSAGE(STATUS "Symbols (liblldb): exporting all symbols from the lldb namespace") + message(STATUS "Symbols (liblldb): exporting all symbols from the lldb namespace") add_llvm_symbol_exports(liblldb ${CMAKE_CURRENT_SOURCE_DIR}/liblldb.exports) elseif (NOT LLDB_EXPORT_ALL_SYMBOLS_EXPORTS_FILE) # Don't use an explicit export. Instead, tell the linker to export all symbols. - MESSAGE(STATUS "Symbols (liblldb): exporting all symbols from the lldb and lldb_private namespaces") - MESSAGE(WARNING "Private LLDB symbols frequently change and no API stability is guaranteed. " + message(STATUS "Symbols (liblldb): exporting all symbols from the lldb and lldb_private namespaces") + message(WARNING "Private LLDB symbols frequently change and no API stability is guaranteed. " "Only the SB API is guaranteed to be stable.") add_llvm_symbol_exports(liblldb ${CMAKE_CURRENT_SOURCE_DIR}/liblldb-private.exports) else () - MESSAGE(STATUS "Symbols (liblldb): exporting all symbols specified in the exports " + message(STATUS "Symbols (liblldb): exporting all symbols specified in the exports " " file '${LLDB_EXPORT_ALL_SYMBOLS_EXPORTS_FILE}'") - MESSAGE(WARNING "Private LLDB symbols frequently change and no API stability is guaranteed. " + message(WARNING "Private LLDB symbols frequently change and no API stability is guaranteed. " "Only the SB API is guaranteed to be stable.") add_llvm_symbol_exports(liblldb "${LLDB_EXPORT_ALL_SYMBOLS_EXPORTS_FILE}") endif() elseif (LLDB_EXPORT_ALL_SYMBOLS) - MESSAGE(STATUS "Symbols (liblldb): exporting all symbols from the lldb and lldb_private namespaces") + message(STATUS "Symbols (liblldb): exporting all symbols from the lldb and lldb_private namespaces") # Pull out the various lldb libraries linked into liblldb, these will be used # when looking for symbols to extract. We ignore most plugin libraries here, _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits