[Lldb-commits] [PATCH] D138315: [lldb] Introduce dwim-print command

2022-11-29 Thread Dave Lee via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG185d4964a158: [lldb] Introduce dwim-print command (authored by kastiglione). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138315/new/ https://reviews.llvm.

[Lldb-commits] [PATCH] D138315: [lldb] Introduce dwim-print command

2022-11-28 Thread Dave Lee via Phabricator via lldb-commits
kastiglione updated this revision to Diff 478419. kastiglione added a comment. Address @labath's feedback Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138315/new/ https://reviews.llvm.org/D138315 Files: lldb/include/lldb/Core/Debugger.h lldb/

[Lldb-commits] [PATCH] D138315: [lldb] Introduce dwim-print command

2022-11-28 Thread Pavel Labath via Phabricator via lldb-commits
labath accepted this revision. labath added inline comments. This revision is now accepted and ready to land. Comment at: lldb/source/Commands/CommandObjectDWIMPrint.cpp:65 +Target *target_ptr = exe_ctx.GetTargetPtr(); +Target &target = target_ptr ? *target_ptr : GetDummy

[Lldb-commits] [PATCH] D138315: [lldb] Introduce dwim-print command

2022-11-18 Thread Dave Lee via Phabricator via lldb-commits
kastiglione updated this revision to Diff 476550. kastiglione added a comment. Add license in .cpp too. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138315/new/ https://reviews.llvm.org/D138315 Files: lldb/include/lldb/Core/Debugger.h lldb/in

[Lldb-commits] [PATCH] D138315: [lldb] Introduce dwim-print command

2022-11-18 Thread Dave Lee via Phabricator via lldb-commits
kastiglione updated this revision to Diff 476549. kastiglione added a comment. Add license. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138315/new/ https://reviews.llvm.org/D138315 Files: lldb/include/lldb/Core/Debugger.h lldb/include/lldb/l

[Lldb-commits] [PATCH] D138315: [lldb] Introduce dwim-print command

2022-11-18 Thread Arthur Eubanks via Phabricator via lldb-commits
aeubanks added inline comments. Comment at: lldb/source/Commands/CommandObjectDWIMPrint.h:1 +#include "lldb/Interpreter/CommandObject.h" + license? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138315/new/ https:/

[Lldb-commits] [PATCH] D138315: [lldb] Introduce dwim-print command

2022-11-18 Thread Dave Lee via Phabricator via lldb-commits
kastiglione created this revision. kastiglione added reviewers: JDevlieghere, aprantl, jingham, labath, werat, cmtice. Herald added a project: All. kastiglione requested review of this revision. Herald added a project: LLDB. Herald added a subscriber: lldb-commits. Implements `dwim-print`, a prin