<div dir="ltr">Yeah, just a quick note about ds2, this is the current status of supported platforms:<div>* Linux (x86/x86_64/arm/aarch64), used for debugging processes on "classic" linux (Ubuntu, centOS, etc), Android, as well as Tizen. </div><div>* Windows (x86/x86_64/arm), used to debug processes on Windows Desktop, Windows Phone, and even on Xbox. Aarch64 support on Windows hasn't been needed so far so we didn't do it, but it would be trivial to add.</div><div>* There is also some support for macOS debugging but some features are still missing there. I'm planning on working on that in the next couple of months, at least for x86 and x86_64 targets.</div><div><br></div><div>We also run the lldb test suite by substituting the default debug server with ds2 and we pass the vast majority of tests there. (A lot of tests are broken or flaky regardless of which debug server is used unfortunately.)<br><div><br></div><div>If people in the lldb community are interested in unifying LLDB's debug servers and have a single small, self-contained binary, we can discuss what would be required to go in that direction.</div><div><br></div><div>Jim pointed out some embedded Apple targets that require debugserver currently. Aren't all these targets based on Darwin? I imagine if we have Darwin-x86_64 fully supported in ds2, adding specific architectures there would a reasonable amount of work.</div><div><br></div><div class="gmail_quote"><div dir="ltr">On Fri, Dec 1, 2017 at 4:13 PM Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">There was also some discussion from some Facebook people about using their own in-house ds2.  It sounds like a large effort, but IIUC it's compatible with debugserver, and supports more platforms than lldb-server.  <div><br></div><div>+stephane and saleem since they know more about it.</div><div><br></div><div>Actually, +greg too since I guess he now works at Facebook.</div></div><div dir="ltr"><div><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Dec 1, 2017 at 4:06 PM Jim Ingham <<a href="mailto:jingham@apple.com" target="_blank">jingham@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">If somebody finds themselves ambitious to work on debugserver, they should divert that energy to porting lldb-server to macOS IMHO.<br>
<br>
Jim<br>
<br>
<br>
> On Dec 1, 2017, at 4:04 PM, Jim Ingham <<a href="mailto:jingham@apple.com" target="_blank">jingham@apple.com</a>> wrote:<br>
><br>
> Yes, we don't use llvm code in debugserver.  It doesn't actually use any lldb classes either, it's its own standalone thing.<br>
><br>
> Jim<br>
><br>
><br>
>> On Dec 1, 2017, at 4:01 PM, Zachary Turner via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>> wrote:<br>
>><br>
>> I don't recall, is there a hard restriction on debugserver not being allowed to use llvm code?  Because YAML is a superset of JSON<br>
>><br>
>> On Fri, Dec 1, 2017 at 3:36 PM Davide Italiano via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>> wrote:<br>
>> Also, FWIW, this is blatantly not the correct way of using assertions.<br>
>> It should either be assert(false && "patatino") or the more canonical<br>
>> llvm_unreachable("patatino").<br>
>> Relying on the implicit conversion from string to bool seems risky<br>
>> (and not really useful).<br>
>><br>
>> On Fri, Dec 1, 2017 at 3:29 PM, Vedant Kumar via lldb-commits<br>
>> <<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>> wrote:<br>
>>> Author: vedantk<br>
>>> Date: Fri Dec  1 15:29:04 2017<br>
>>> New Revision: 319596<br>
>>><br>
>>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=319596&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=319596&view=rev</a><br>
>>> Log:<br>
>>> Fix warnings in JSON.cpp, NFC<br>
>>><br>
>>> These asserts are no-ops, and are supplanted by -Wcovered-switch.<br>
>>><br>
>>> Modified:<br>
>>>    lldb/trunk/tools/debugserver/source/JSON.cpp<br>
>>><br>
>>> Modified: lldb/trunk/tools/debugserver/source/JSON.cpp<br>
>>> URL: <a href="http://llvm.org/viewvc/llvm-project/lldb/trunk/tools/debugserver/source/JSON.cpp?rev=319596&r1=319595&r2=319596&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/lldb/trunk/tools/debugserver/source/JSON.cpp?rev=319596&r1=319595&r2=319596&view=diff</a><br>
>>> ==============================================================================<br>
>>> --- lldb/trunk/tools/debugserver/source/JSON.cpp (original)<br>
>>> +++ lldb/trunk/tools/debugserver/source/JSON.cpp Fri Dec  1 15:29:04 2017<br>
>>> @@ -58,7 +58,6 @@ uint64_t JSONNumber::GetAsUnsigned() con<br>
>>>   case DataType::Double:<br>
>>>     return (uint64_t)m_data.m_double;<br>
>>>   }<br>
>>> -  assert("Unhandled data type");<br>
>>> }<br>
>>><br>
>>> int64_t JSONNumber::GetAsSigned() const {<br>
>>> @@ -70,7 +69,6 @@ int64_t JSONNumber::GetAsSigned() const<br>
>>>   case DataType::Double:<br>
>>>     return (int64_t)m_data.m_double;<br>
>>>   }<br>
>>> -  assert("Unhandled data type");<br>
>>> }<br>
>>><br>
>>> double JSONNumber::GetAsDouble() const {<br>
>>> @@ -82,7 +80,6 @@ double JSONNumber::GetAsDouble() const {<br>
>>>   case DataType::Double:<br>
>>>     return m_data.m_double;<br>
>>>   }<br>
>>> -  assert("Unhandled data type");<br>
>>> }<br>
>>><br>
>>> void JSONNumber::Write(std::ostream &s) {<br>
>>><br>
>>><br>
>>> _______________________________________________<br>
>>> lldb-commits mailing list<br>
>>> <a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a><br>
>>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br>
>> _______________________________________________<br>
>> lldb-commits mailing list<br>
>> <a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a><br>
>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br>
>> _______________________________________________<br>
>> lldb-commits mailing list<br>
>> <a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a><br>
>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br>
><br>
<br>
</blockquote></div></div></div></blockquote></div></div></div><div dir="ltr">-- <br></div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">-- <br>Stephane Sezer</div></div>