mgorny created this revision.
mgorny added reviewers: labath, krytarowski, emaste.
mgorny requested review of this revision.
https://reviews.llvm.org/D110020
Files:
lldb/source/Plugins/Process/gdb-remote/GDBRemoteRegisterContext.cpp
lldb/source/Plugins/Process/gdb-remote/GDBRemoteRegisterCont
mgorny updated this revision to Diff 373410.
mgorny added a comment.
Add a helper function. Test the weird invalidate_regs expansion logic.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109906/new/
https://reviews.llvm.org/D109906
Files:
lldb/unittests/Process/Utility/CMakeLists.txt
mgorny updated this revision to Diff 373411.
mgorny added a comment.
Eliminate unused vars.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109906/new/
https://reviews.llvm.org/D109906
Files:
lldb/unittests/Process/Utility/CMakeLists.txt
lldb/unittests/Process/Utility/DynamicRegister
mgorny updated this revision to Diff 373412.
mgorny retitled this revision from "[lldb] [DynamicRegisterInfo] Update
RegisterInfo with copy of value_regs/invalidate_regs" to "[lldb]
[DynamicRegisterInfo] Replace value_regs/invalidate_regs in AddRegister()".
mgorny edited the summary of this revis
mgorny created this revision.
mgorny added reviewers: labath, krytarowski, emaste.
mgorny requested review of this revision.
Add a convenience method to add supplementary registers that takes care
of adding invalidate_regs to all (potentially) overlapping registers.
https://reviews.llvm.org/D110
mgorny added a comment.
Currently it's still using process plugin regnums in
value_regs/invalidate_regs; I'm going to try changing that next.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D110023/new/
https://reviews.llvm.org/D110023
___
lldb
mgorny created this revision.
mgorny added reviewers: labath, krytarowski, emaste.
mgorny requested review of this revision.
Refactor ProcessGDBRemote::GetGDBServerRegisterInfo() and helper
routines to collect remote registers into a local std::vector rather
than adding them straight into DynamicR
mgorny added a comment.
@labath, please tell me if you like this design. If it's ok, I'm going to
convert the other routine to it, then move the conversion to
`DynamicRegisterInfo` into a common function.
(also I haven't reformatted the code yet)
CHANGES SINCE LAST ACTION
https://reviews.ll
mgorny updated this revision to Diff 373426.
mgorny added a comment.
clang-format
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109879/new/
https://reviews.llvm.org/D109879
Files:
lldb/source/Plugins/Process/Utility/DynamicRegisterInfo.cpp
lldb/source/Plugins/Process/Utility/Dynami
mgorny updated this revision to Diff 373427.
mgorny added a comment.
clang-format
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D110023/new/
https://reviews.llvm.org/D110023
Files:
lldb/source/Plugins/Process/Utility/DynamicRegisterInfo.cpp
lldb/source/Plugins/Process/Utility/Dynami
mgorny created this revision.
mgorny added reviewers: labath, krytarowski, emaste.
mgorny requested review of this revision.
Switch the gdb-remote client logic to use local (LLDB) register numbers
in value_regs/invalidate_regs rather than remote regnos. This involves
translating regnos received f
11 matches
Mail list logo