labath added a comment.

In D108831#2999227 <https://reviews.llvm.org/D108831#2999227>, @mgorny wrote:

> Another question: should we aim to remove the 'redundant afterwards' bits 
> from lldb-server, and fill them in from client side entirely? I don't think 
> our process plugins really need knowledge of EAX etc., and it would reduce 
> the duplication a fair bit.

I think that would be great. Besides the deduplication aspect, this will also 
decrease the chances of some gdb compatibility bug creeping in. I might 
consider waiting for one release, just in case someone wants to connect a newer 
lldb-server to an old lldb.

/And/, I would actually like to go further than that, and remove these register 
definitions from core file (&minidump, etc) plugins too. Since eax/ax/ah/al can 
be extracted from rax in a generic way, I don't see a reason for each plugin to 
have to redefine those (or forget doing like, like is the case with the windows 
plugin). This is the point where this logic stops being gdb-remote-specific. 
There are no compatibility issues here, though the process itself might be 
trickier, depending on how much the plugins are assuming a particular register 
order, etc.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108831/new/

https://reviews.llvm.org/D108831

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to