[llvm] Improve error response message parsing for DAP evaluate requests. (PR #168430)

Stephen Tozer via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 18 03:29:35 PST 2025


SLTozer wrote:

> Merging to unblock ci but I do wonder if we should share parts of the Python client code so we don’t miss some things like this.

There could be benefits to having a shared implementation, but my gut says that it could also be more trouble than it's worth - although the debug adapter protocol is "generic", in practice it still seems to require some specialization per-debugger and per-client (from the small sample size I've observed), and I suspect the "Dexter version" might end up with significant deviations based on the needs of Dexter and the behaviour of other debug adapters. With that said, that's just my first impression - I'll take the time to compare the implementations in more detail later, since I agree it would simplify ongoing maintenance if the code could be shared.

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


More information about the llvm-commits mailing list