[Lldb-commits] [PATCH] D130342: [LLDB][RISCV] Add riscv register stuff

2022-08-10 Thread David Spickett via Phabricator via lldb-commits
DavidSpickett accepted this revision. DavidSpickett added a comment. This revision is now accepted and ready to land. > we can add them back if required in the future. Sounds good to me. Certainly SVE for Arm required a bunch of changes, a lot easier to do it later. This LGTM, but: [LLDB][RI

[Lldb-commits] [PATCH] D130342: [LLDB][RISCV] Add riscv register stuff

2022-08-10 Thread Emmmer S via Phabricator via lldb-commits
Emmmer updated this revision to Diff 451442. Emmmer marked an inline comment as done. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130342/new/ https://reviews.llvm.org/D130342 Files: lldb/source/Host/common/HostInfoBase.cpp lldb/source/Plugins/Process/Linux/CMakeLists.txt lldb/sou

[Lldb-commits] [PATCH] D130342: [LLDB][RISCV] Add riscv register stuff

2022-08-10 Thread Emmmer S via Phabricator via lldb-commits
Emmmer updated this revision to Diff 451441. Emmmer retitled this revision from "[LLDB][RISCV] Add register stuff and make breakpoint work" to "[LLDB][RISCV] Add riscv register stuff". Emmmer edited the summary of this revision. Emmmer added a comment. Address review comments: - Split the softwa