<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.<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">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>