[Lldb-commits] [PATCH] D62887: Update the thread list before setting stop reasons with an OS plugin

Greg Clayton via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Jun 7 14:06:16 PDT 2019


clayborg added inline comments.


================
Comment at: source/Target/Process.cpp:3037-3041
+  // Somebody might have gotten threads before now, but we need to force the
+  // update after we've loaded the OperatingSystem plugin or it won't get a
+  // chance to process the threads.
+  m_thread_list.Clear();
+  UpdateThreadListIfNeeded();
----------------
Should this be in the above if statement?

```
if (!m_os_up) {
    LoadOperatingSystemPlugin(false);
    // Somebody might have gotten threads before now, but we need to force the
    // update after we've loaded the OperatingSystem plugin or it won't get a
    // chance to process the threads.
    m_thread_list.Clear();
    UpdateThreadListIfNeeded();
}
```

And if we do this in the if statement, do we need to clear the m_thread_list?


Repository:
  rLLDB LLDB

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62887/new/

https://reviews.llvm.org/D62887





More information about the lldb-commits mailing list