Remove SetResumeState from POSIXThread::RefreshStateAfterStop
authorEd Maste <emaste@freebsd.org>
Wed, 24 Jun 2015 16:39:22 +0000 (16:39 +0000)
committerEd Maste <emaste@freebsd.org>
Wed, 24 Jun 2015 16:39:22 +0000 (16:39 +0000)
With the removal of ProcessLinux in r240543 this code is used only on
FreeBSD. FreeBSD isn't affected by whichever issue originally prompted
the addition of SetResumeState, so just remove it.

As discussed on the mailing list (and mentioned in a FIXME comment)
it shouldn't be called there.

llvm-svn: 240550

lldb/source/Plugins/Process/FreeBSD/POSIXThread.cpp

index 5294bbe..854796f 100644 (file)
@@ -95,7 +95,6 @@ POSIXThread::GetMonitor()
     return process.GetMonitor();
 }
 
-// Overridden by FreeBSDThread; this is used only on Linux.
 void
 POSIXThread::RefreshStateAfterStop()
 {
@@ -112,11 +111,6 @@ POSIXThread::RefreshStateAfterStop()
         const bool force = false;
         GetRegisterContext()->InvalidateIfNeeded (force);
     }
-    // FIXME: This should probably happen somewhere else.
-    SetResumeState(eStateRunning, true);
-    Log *log (ProcessPOSIXLog::GetLogIfAllCategoriesSet (POSIX_LOG_THREAD));
-    if (log)
-        log->Printf ("POSIXThread::%s (tid = %" PRIi64 ") setting thread resume state to running", __FUNCTION__, GetID());
 }
 
 const char *