[Lldb-commits] [lldb] [lldb][RPC] Upstream lldb-rpc-gen tool (PR #138031)

[Chelsea Cassanova via lldb-commits]

chelcassanova wrote:

> I'd be alright with it going in a follow-up. That being said, it would be easy to kick this can down the road. Let's figure out a way to make this happen without blocking this PR.

I think that this should be addressed in a follow-up PR, mainly because I think changes to remove this list might end up touching other parts of the SB API. I'd rather that the tool land and then follow up on this as it's a larger issue.

> Yes. Alternatively, let's have our internal clients move away from these so we can remove it altogether 😄

Correct me if I'm wrong but we might've already done this? Either way this is the smallest exception list so I think I can address this still in this patch.

> Perhaps, but all of this can be done in a follow-up (unless requested by another reviewer). I don't think it's essential this change right now.

Agreed.

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