================
@@ -247,10 +247,8 @@ bool
ProcessFreeBSDKernelCore::DoUpdateThreadList(ThreadList &old_thread_list,
// https://cgit.freebsd.org/src/tree/sys/sys/param.h
constexpr size_t fbsd_maxcomlen = 19;
- // Iterate through a linked list of all processes. New processes are added
- // to the head of this list. Which means that earlier PIDs are actually at
- // the end of the list, so we have to walk it backwards. First collect all
- // the processes in the list order.
+ // Iterate through a linked list of all processes then order incrementally
+ // by pid.
----------------
DavidSpickett wrote:
Add something to make it clear that there is no expected order.
"Though new processes are added to the head of this list, process objects may
be reused as well. So we cannot rely on it being in a particular order."
https://github.com/llvm/llvm-project/pull/187976
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
