Author: Jonas Devlieghere Date: 2024-05-07T13:12:22-07:00 New Revision: dad11097096c05564758e539f9f03ef883365fdd
URL: https://github.com/llvm/llvm-project/commit/dad11097096c05564758e539f9f03ef883365fdd DIFF: https://github.com/llvm/llvm-project/commit/dad11097096c05564758e539f9f03ef883365fdd.diff LOG: [lldb] Reinstate lldb-sbapi-dwarf-enums target (NFC) (#91390) Alex pointed out in #91254 that we only need the custom target if we had more than one target depending on it. This isn't the case upstream, but on our downstream fork, we have a second dependency. Reintroduce the target so that everything can depend on that, without the single-dependency foot-gun. Added: Modified: lldb/source/API/CMakeLists.txt Removed: ################################################################################ diff --git a/lldb/source/API/CMakeLists.txt b/lldb/source/API/CMakeLists.txt index 798a92874f13d..aa31caddfde3a 100644 --- a/lldb/source/API/CMakeLists.txt +++ b/lldb/source/API/CMakeLists.txt @@ -20,7 +20,7 @@ if(LLDB_ENABLE_LUA) set(lldb_lua_wrapper ${lua_bindings_dir}/LLDBWrapLua.cpp) endif() -# Target to generate SBLanguages.h from Dwarf.def. +# Generate SBLanguages.h from Dwarf.def. set(sb_languages_file ${CMAKE_CURRENT_BINARY_DIR}/../../include/lldb/API/SBLanguages.h) add_custom_command( @@ -33,6 +33,8 @@ add_custom_command( DEPENDS ${LLVM_MAIN_INCLUDE_DIR}/llvm/BinaryFormat/Dwarf.def WORKING_DIRECTORY ${LLVM_LIBRARY_OUTPUT_INTDIR} ) +add_custom_target(lldb-sbapi-dwarf-enums + DEPENDS ${sb_languages_file}) add_lldb_library(liblldb SHARED ${option_framework} SBAddress.cpp @@ -113,7 +115,9 @@ add_lldb_library(liblldb SHARED ${option_framework} SystemInitializerFull.cpp ${lldb_python_wrapper} ${lldb_lua_wrapper} - ${sb_languages_file} + + DEPENDS + lldb-sbapi-dwarf-enums LINK_LIBS lldbBreakpoint _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits