================
@@ -183,13 +183,21 @@ bool ProcessFreeBSDKernel::DoUpdateThreadList(ThreadList 
&old_thread_list,
     // from FreeBSD sys/param.h
     constexpr size_t fbsd_maxcomlen = 19;
 
-    // iterate through a linked list of all processes
-    // allproc is a pointer to the first list element, p_list field
-    // (found at offset_p_list) specifies the next element
+    // pid is in reverse order. Order it incrementally
----------------
DavidSpickett wrote:

Please keep some of the context from the previous comment, like:
```
// 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.
```

https://github.com/llvm/llvm-project/pull/178306
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to