DavidSpickett marked an inline comment as done.
DavidSpickett added inline comments.
================
Comment at: lldb/source/Plugins/Process/Linux/NativeProcessLinux.cpp:1357
+
+ if (error.Fail())
+ return error;
----------------
omjavaid wrote:
> ptrace request is a success if number of tags requested is not equal to no of
> tags read? If not then this and following condition may be redundant.
Well ptracewrapper doesn't check the iovec, but I'll check the kernel source to
see if it's actually possible for it to fail that way.
================
Comment at:
lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerLLGS.cpp:3578
+
+ response.PutChar('m');
+ response.PutBytesAsRawHex8(tags.data(), tags.size());
----------------
omjavaid wrote:
> Just curious response starts with 'm'. Whats the design need for using m in
> qMemTags response?
This is for future multi part replies ala qfThreadInfo
(https://sourceware.org/gdb/current/onlinedocs/gdb/General-Query-Packets.html#General-Query-Packets).
I'll add a comment with this too.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95601/new/
https://reviews.llvm.org/D95601
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
