Skip to content

Commit 26f8029

Browse files
committed
[lldb] Reinstate lldb-sbapi-dwarf-enums target (NFC) (llvm#91390)
Alex pointed out in llvm#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. (cherry picked from commit dad1109)
1 parent 18013b9 commit 26f8029

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

lldb/source/API/CMakeLists.txt

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ if(LLDB_ENABLE_LUA)
3030
set(lldb_lua_wrapper ${lua_bindings_dir}/LLDBWrapLua.cpp)
3131
endif()
3232

33-
# Target to generate SBLanguages.h from Dwarf.def.
33+
# Generate SBLanguages.h from Dwarf.def.
3434
set(sb_languages_file
3535
${CMAKE_CURRENT_BINARY_DIR}/../../include/lldb/API/SBLanguages.h)
3636
add_custom_command(
@@ -43,6 +43,8 @@ add_custom_command(
4343
DEPENDS ${LLVM_MAIN_INCLUDE_DIR}/llvm/BinaryFormat/Dwarf.def
4444
WORKING_DIRECTORY ${LLVM_LIBRARY_OUTPUT_INTDIR}
4545
)
46+
add_custom_target(lldb-sbapi-dwarf-enums
47+
DEPENDS ${sb_languages_file})
4648

4749
add_lldb_library(liblldb SHARED ${option_framework}
4850
SBAddress.cpp
@@ -121,7 +123,9 @@ add_lldb_library(liblldb SHARED ${option_framework}
121123
SystemInitializerFull.cpp
122124
${lldb_python_wrapper}
123125
${lldb_lua_wrapper}
124-
${sb_languages_file}
126+
127+
DEPENDS
128+
lldb-sbapi-dwarf-enums
125129

126130
LINK_LIBS
127131
lldbBreakpoint

0 commit comments

Comments
 (0)