<div dir="ltr">+1<div><br></div><div>This all sounds in line with the expectations we've laid out on the mailing list in the past for platform/language support. <br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 9, 2021 at 12:24 AM Pavel Labath via lldb-dev <<a href="mailto:lldb-dev@lists.llvm.org">lldb-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Hi all,<br>
<br>
I propose to remove support for linux mips debugging. This basically <br>
amounts to deleting <br>
source/Plugins/Process/Linux/NativeRegisterContextLinux_mips64.{cpp,h}. <br>
My reasons for doing that are:<br>
<br>
- This code is unmaintained (last non-mechanical change was in 2017) and <br>
untested (no public buildbots), so we don't know if even basic <br>
functionality works, or if it indeed builds.<br>
<br>
- At the same, it is carrying a lot of technical debt, which is leaking <br>
out of the mips-specific files, and interfering with other development <br>
efforts. The last instance of this is D96766, which is adding FreeBSD <br>
mips support, but needs to work around linux specific knowledge leaking <br>
into supposedly generic code. This one should be fixable relatively <br>
easily (these days we already have precedents for similar things in x86 <br>
and arm code), but it needs someone who is willing to do that.<br>
<br>
But that is not all. To support mips, we introduced two new fields into <br>
the RegisterInfo struct (dynamic_size_dwarf_{expr_bytes,len}). These are <br>
introducing a lot of clutter in all our RegisterInfo definitions (which <br>
we have **a lot** of) and are not really consistent with the long term <br>
vision of the gdb-remote protocol usage in lldb. These days, we have a <br>
different mechanism for this (added to support a similar feature in <br>
arm), it would be better to implement this feature in terms of that. I <br>
would tout this (removal of these fields) as the main benefit of <br>
dropping mips support.<br>
<br>
So, unless someone willing to address these issues (I'm happy to provide <br>
support where I can), I propose we drop mips support. Generic mips <br>
support will remain (and hopefully be better tested) thanks to the <br>
FreeBSD mips port, so re-adding mips support should be a matter of <br>
reimplementing the linux bits.<br>
<br>
regards,<br>
Pavel<br>
_______________________________________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev</a><br>
</blockquote></div>