ramana-nvr added a comment.

In https://reviews.llvm.org/D48868#1156416, @jasonmolenda wrote:

>   If jThreadsInfo isn't implemented, we fall back to looking for the 
> thread-pcs: and threads: key-value pairs in the stop packet that we received. 
>
> We look for the threads: list and if it is present, we call 
> UpdateThreadIDsFromStopReplyThreadsValue which clears m_thread_pcs again (bug 
> #2) and then adds any threads listed to the m_thread_ids.


Yes, we shouldn't be clearing the m_thread_pcs in 
UpdateThreadIDsFromStopReplyThreadsValue() otherwise we would loose the thread 
PCs populated from stop reply thread-pcs: list just before the 
UpdateThreadIDsFromStopReplyThreadsValue() call.

The other m_thread_pcs.clear() in UpdateThreadIDList() is redundant as we are 
anyway clearing m_thread_pcs in UpdateThreadPCsFromStopReplyThreadsValue().


https://reviews.llvm.org/D48868



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

Reply via email to