[Lldb-commits] [PATCH] D147385: [lldb] Add fixits for "frame variable"

2023-04-03 Thread Adrian Prantl via Phabricator via lldb-commits
aprantl added a comment. That looks like a nice quality of life improvement! Personally I would lean towards replicating the expr behavior, though I don't know how much work that would be. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147385/new/

[Lldb-commits] [PATCH] D147385: [lldb] Add fixits for "frame variable"

2023-04-01 Thread Dave Lee via Phabricator via lldb-commits
kastiglione updated this revision to Diff 510280. kastiglione added a comment. Change --apply-fixits take an argument Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147385/new/ https://reviews.llvm.org/D147385 Files: lldb/include/lldb/Interpreter

[Lldb-commits] [PATCH] D147385: [lldb] Add fixits for "frame variable"

2023-04-01 Thread Dave Lee via Phabricator via lldb-commits
kastiglione created this revision. kastiglione added reviewers: aprantl, jingham, JDevlieghere, augusto2112. Herald added a project: All. kastiglione requested review of this revision. Herald added a project: LLDB. Herald added a subscriber: lldb-commits. The `frame variable` command can handle mi