Skip to content

Conversation

Bertik23
Copy link
Contributor

@llvmbot
Copy link
Member

llvmbot commented Sep 15, 2025

@llvm/pr-subscribers-llvm-support

Author: Bertik23 (Bertik23)

Changes

Fixes warning recognized here https://github.com/llvm/llvm-project/pull/157885/files/7d06b3db4f8f7045ad681d1dd017f920bcc115d3#r2348174808 by restoring code from e24a7bb


Full diff: https://github.com/llvm/llvm-project/pull/158666.diff

1 Files Affected:

  • (modified) llvm/lib/Support/LSP/Transport.cpp (+1-1)
diff --git a/llvm/lib/Support/LSP/Transport.cpp b/llvm/lib/Support/LSP/Transport.cpp
index e71f17701636b..895ed62495282 100644
--- a/llvm/lib/Support/LSP/Transport.cpp
+++ b/llvm/lib/Support/LSP/Transport.cpp
@@ -120,7 +120,7 @@ bool MessageHandler::onReply(llvm::json::Value Id,
   // mapping and erase it.
   ResponseHandlerTy ResponseHandler;
   {
-    std::lock_guard<std::mutex> responseHandlersLock(ResponseHandlerTy);
+    std::lock_guard<std::mutex> ResponseHandlersLock(ResponseHandlersMutex);
     auto It = ResponseHandlers.find(debugString(Id));
     if (It != ResponseHandlers.end()) {
       ResponseHandler = std::move(It->second);

@Bertik23
Copy link
Contributor Author

Fixed in #160010

@Bertik23 Bertik23 closed this Sep 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants