[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