aboutsummaryrefslogtreecommitdiff
path: root/clang/lib/Analysis/ThreadSafetyCommon.cpp
diff options
context:
space:
mode:
authorJonas Devlieghere <jonas@devlieghere.com>2025-08-25 16:18:21 -0700
committerGitHub <noreply@github.com>2025-08-25 16:18:21 -0700
commit1ba8b36fef84bedb0a657b570076ec1a47e9061d (patch)
tree5da2755a9d215d7eade3b3ffe1c332ba88ce7ac7 /clang/lib/Analysis/ThreadSafetyCommon.cpp
parent508ef1796cf3fb590c79e2c76bf3339d3e4a8739 (diff)
downloadllvm-1ba8b36fef84bedb0a657b570076ec1a47e9061d.zip
llvm-1ba8b36fef84bedb0a657b570076ec1a47e9061d.tar.gz
llvm-1ba8b36fef84bedb0a657b570076ec1a47e9061d.tar.bz2
[lldb] Adopt JSONTransport in the MCP Server (Reland) (#155322)
This PR adopts JSONTransport in the MCP server implementation. It required a slight change in design in the relationship between the two server classes. Previously, these two had an "is-a" connection, while now they have a "has-a" connection. The "generic" protocol server in Protocol/MCP now operates using a single connection (Transport). This matches the design in DAP where each DAP instance has its own connection. The protocol server in Plugins still supports multiple clients and creates a new server instance for each connection. I believe the new design makes sense in the long term (as proved by DAP) and allows us to make the server stateful if we choose to do so. There's no reason that multiple client support can't live in the generic protocol library, but for now I kept it in ProtocolServerMCP to avoid creating unnecessary abstractions. This is a reland of #155034 but with significant changes to the tests. The unit tests now test the generic server implementation, which matches the original intent. This also means the test are now single threaded and therefore fully deterministic using the MainLoop.
Diffstat (limited to 'clang/lib/Analysis/ThreadSafetyCommon.cpp')
0 files changed, 0 insertions, 0 deletions