jarin added inline comments.
================ Comment at: lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerLLGS.cpp:2819 + response.Printf("<feature>"); + const int registersCount = 128; + for (int reg_index = 0; reg_index < registersCount; reg_index++) { ---------------- As discussed offline, this should not be hard-coded, register context provides register count. ================ Comment at: lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerLLGS.cpp:2825 + response.Printf( + "<reg name=\"%s\" bitsize=\"%u\" offset=\"%u\" regnum=\"%d\" />", + reg_info->name, reg_info->byte_size * 8, reg_info->byte_offset, ---------------- Fill all the relevant fields, please (everything that the qRegisterInfo handler provides). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74217/new/ https://reviews.llvm.org/D74217 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits