Skip to content

Commit 2c02c4e

Browse files
JDevlieghereadrian-prantl
authored andcommitted
[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) (cherry picked from commit 26f8029)
1 parent 22e9d1d commit 2c02c4e

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
@@ -123,7 +125,9 @@ add_lldb_library(liblldb SHARED ${option_framework}
123125
SystemInitializerFull.cpp
124126
${lldb_python_wrapper}
125127
${lldb_lua_wrapper}
126-
${sb_languages_file}
128+
129+
DEPENDS
130+
lldb-sbapi-dwarf-enums
127131

128132
LINK_LIBS
129133
lldbBreakpoint

0 commit comments

Comments
 (0)