[Lldb-commits] [lldb] r234366 - Make sure the OperatingSystem plug-ins will work on linux as well. This goes with my previous fix with revision 234364.

Greg Clayton gclayton at apple.com
Tue Apr 7 15:23:53 PDT 2015


Author: gclayton
Date: Tue Apr  7 17:23:53 2015
New Revision: 234366

URL: http://llvm.org/viewvc/llvm-project?rev=234366&view=rev
Log:
Make sure the OperatingSystem plug-ins will work on linux as well. This goes with my previous fix with revision 234364.


Modified:
    lldb/trunk/source/Plugins/Process/POSIX/POSIXThread.cpp

Modified: lldb/trunk/source/Plugins/Process/POSIX/POSIXThread.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/Process/POSIX/POSIXThread.cpp?rev=234366&r1=234365&r2=234366&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/Process/POSIX/POSIXThread.cpp (original)
+++ lldb/trunk/source/Plugins/Process/POSIX/POSIXThread.cpp Tue Apr  7 17:23:53 2015
@@ -512,7 +512,10 @@ POSIXThread::BreakNotify(const ProcessMe
     if (bp_site)
     {
         lldb::break_id_t bp_id = bp_site->GetID();
-        if (bp_site->ValidForThisThread(this))
+        // If we have an operating system plug-in, we might have set a thread specific breakpoint using the
+        // operating system thread ID, so we can't make any assumptions about the thread ID so we must always
+        // report the breakpoint regardless of the thread.
+        if (bp_site->ValidForThisThread(this) || thread.GetProcess()->GetOperatingSystem () != NULL)
             SetStopInfo (StopInfo::CreateStopReasonWithBreakpointSiteID(*this, bp_id));
         else
         {





More information about the lldb-commits mailing list