================ @@ -3144,16 +3144,17 @@ void Process::CompleteAttach() { } } - if (!m_os_up) { + if (!m_os_up) LoadOperatingSystemPlugin(false); - if (m_os_up) { - // 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(); - } + + if (m_os_up) { ---------------- JDevlieghere wrote:
My first instinct here was "should this be an else" before I realized that `LoadOperatingSystemPlugin` probably initializes `m_os_up`. Might be wroth a comment. https://github.com/llvm/llvm-project/pull/146441 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits