This is an archive of the discontinued LLVM Phabricator instance.

Differential D36948

lldb-argdumper doesn't need lldbCore.
ClosedPublic

Authored by brucem on Aug 20 2017, 11:24 PM.

Details

Reviewers
beanz
zturner
labath
Commits
rG416bc3d70ab7: lldb-argdumper doesn't need lldbCore.
rLLDB311399: lldb-argdumper doesn't need lldbCore.
rL311399: lldb-argdumper doesn't need lldbCore.
Summary

lldb-argdumper only needs lldbUtility to successfully build and link.

Diff Detail

Repository
rL LLVM

Event Timeline

brucem created this revision.Aug 20 2017, 11:24 PM
Herald added a subscriber: mgorny. · View Herald TranscriptAug 20 2017, 11:24 PM
zturner accepted this revision.Aug 21 2017, 9:33 AM
This revision is now accepted and ready to land.Aug 21 2017, 9:33 AM
Closed by commit rL311399: lldb-argdumper doesn't need lldbCore. (authored by brucem). · Explain WhyAug 21 2017, 3:56 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathSize
lldb/
trunk/
tools/
argdumper/
1 line

Diff 112082

lldb/trunk/tools/argdumper/CMakeLists.txt

include(${LLDB_PROJECT_ROOT}/cmake/LLDBDependencies.cmake) include(${LLDB_PROJECT_ROOT}/cmake/LLDBDependencies.cmake)
add_lldb_tool(lldb-argdumper INCLUDE_IN_FRAMEWORK add_lldb_tool(lldb-argdumper INCLUDE_IN_FRAMEWORK
argdumper.cpp argdumper.cpp
LINK_LIBS LINK_LIBS
lldbCore
lldbUtility lldbUtility
) )