Re: [Lldb-commits] [lldb] r313442 - Fix compatibility with OpenOCD debug stub.

2017-09-18 Thread Vadim Chugunov via lldb-commits
Yes, this works for OpenOCD as well. Thanks! On Mon, Sep 18, 2017 at 4:44 AM, Tamas Berghammer wrote: > Hi Vadim, > > This change broke remote debugging on Linux and Android as for some reason > LLDB sends a qfThreadInfo on those platforms before starting a process (not > sure why, will investi

Re: [Lldb-commits] [lldb] r313442 - Fix compatibility with OpenOCD debug stub.

2017-09-18 Thread Greg Clayton via lldb-commits
That is the reason I asked for the patch to verify "l" was the only thing received before it goes and makes up a fake thread ID... > On Sep 18, 2017, at 4:44 AM, Tamas Berghammer via lldb-commits > wrote: > > Hi Vadim, > > This change broke remote debugging on Linux and Android as for some re

Re: [Lldb-commits] [lldb] r313442 - Fix compatibility with OpenOCD debug stub.

2017-09-18 Thread Tamas Berghammer via lldb-commits
Hi Vadim, This change broke remote debugging on Linux and Android as for some reason LLDB sends a qfThreadInfo on those platforms before starting a process (not sure why, will investigate when I have a bit more time) and lldb-server sends an OK response to it. After your change it will generate a

[Lldb-commits] [lldb] r313442 - Fix compatibility with OpenOCD debug stub.

2017-09-15 Thread Vadim Chugunov via lldb-commits
Author: vadimcn Date: Fri Sep 15 20:53:13 2017 New Revision: 313442 URL: http://llvm.org/viewvc/llvm-project?rev=313442&view=rev Log: Fix compatibility with OpenOCD debug stub. OpenOCD sends register classes as two separate nodes, fixed parser to process both of them. OpenOCD returns "l" in re