kastiglione created this revision.
kastiglione added reviewers: aprantl, JDevlieghere, jingham, labath, werat.
Herald added a subscriber: jeroen.dobbelaere.
Herald added a project: All.
kastiglione requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-comm
aprantl added a comment.
So, one could say that (right now) this patch is NFC for end-users, because
`dwim-print` supports all use-cases that the old `p` alias supported?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D145189/new/
https://reviews.ll
aprantl accepted this revision.
aprantl added a comment.
This revision is now accepted and ready to land.
So from my point of view, this is a good path forward, so assuming that the
other reviewers agree, this LGTM.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://revi
kastiglione added a comment.
> So, one could say that (right now) this patch is NFC for end-users, because
> `dwim-print` supports all use-cases that the old `p` alias supported?
Correct.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D145189/new/
This revision was automatically updated to reflect the committed changes.
Closed by commit rGa00801d94b02: [lldb] Redefine p alias to dwim-print command
(authored by kastiglione).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D145189/new/
https://rev