[Lldb-commits] [lldb] Add option to pass thread ID to thread select command (PR #73596)

Greg Clayton via lldb-commits lldb-commits at lists.llvm.org
Thu Nov 30 12:14:22 PST 2023


================
@@ -1296,10 +1296,11 @@ enum CompletionType {
   eRemoteDiskFileCompletion = (1u << 22),
   eRemoteDiskDirectoryCompletion = (1u << 23),
   eTypeCategoryNameCompletion = (1u << 24),
+  eThreadIDCompletion = (1u << 25),
----------------
clayborg wrote:

> @clayborg Is there anything special about `eCustomCompletion` being last? The comment in 1300-1302 seems to allude to this, but I don't see anything in the code base requiring this nor any other custom completions.

Yes, this is a problem. If there ever is a magic bit it should be something like:
```
eCustomCompletion = (1u << 63) // Pick the last bit in a 64 bit value
```
But this is public API now which causes a problem for reasons I mentioned before. @jimingham any ideas on if we care about adding the thread ID before the `eCustomCompletion` from an API standpoint? lldb-rpc-server is the main thing I worry about here since it sends enums as integers, not as strings which are then converted back into integers.

https://github.com/llvm/llvm-project/pull/73596


More information about the lldb-commits mailing list