Skip to content

Commit dad1109

Browse files
authored
[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.
1 parent c6855ab commit dad1109

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
@@ -20,7 +20,7 @@ if(LLDB_ENABLE_LUA)
2020
set(lldb_lua_wrapper ${lua_bindings_dir}/LLDBWrapLua.cpp)
2121
endif()
2222

23-
# Target to generate SBLanguages.h from Dwarf.def.
23+
# Generate SBLanguages.h from Dwarf.def.
2424
set(sb_languages_file
2525
${CMAKE_CURRENT_BINARY_DIR}/../../include/lldb/API/SBLanguages.h)
2626
add_custom_command(
@@ -33,6 +33,8 @@ add_custom_command(
3333
DEPENDS ${LLVM_MAIN_INCLUDE_DIR}/llvm/BinaryFormat/Dwarf.def
3434
WORKING_DIRECTORY ${LLVM_LIBRARY_OUTPUT_INTDIR}
3535
)
36+
add_custom_target(lldb-sbapi-dwarf-enums
37+
DEPENDS ${sb_languages_file})
3638

3739
add_lldb_library(liblldb SHARED ${option_framework}
3840
SBAddress.cpp
@@ -113,7 +115,9 @@ add_lldb_library(liblldb SHARED ${option_framework}
113115
SystemInitializerFull.cpp
114116
${lldb_python_wrapper}
115117
${lldb_lua_wrapper}
116-
${sb_languages_file}
118+
119+
DEPENDS
120+
lldb-sbapi-dwarf-enums
117121

118122
LINK_LIBS
119123
lldbBreakpoint

0 commit comments

Comments
 (0)