Skip to content

Conversation

@da-viper
Copy link
Contributor

@da-viper da-viper commented Jun 4, 2025

fixes the failed tests on aarch64.

complements #142439

fixes the failed tests on arch64.

complements llvm#142439
@llvmbot
Copy link
Member

llvmbot commented Jun 4, 2025

@llvm/pr-subscribers-lldb

Author: Ebuka Ezike (da-viper)

Changes

fixes the failed tests on arch64.

complements #142439


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

1 Files Affected:

  • (modified) lldb/tools/lldb-dap/EventHelper.cpp (+3-1)
diff --git a/lldb/tools/lldb-dap/EventHelper.cpp b/lldb/tools/lldb-dap/EventHelper.cpp
index 0dd0b61b1e2ae..33bc7c2cbef11 100644
--- a/lldb/tools/lldb-dap/EventHelper.cpp
+++ b/lldb/tools/lldb-dap/EventHelper.cpp
@@ -46,7 +46,9 @@ void SendTargetBasedCapabilities(DAP &dap) {
 
   protocol::Event event;
   event.event = "capabilities";
-  event.body = llvm::json::Object{{"supportsStepInTargetsRequest", false}};
+  event.body = llvm::json::Object{
+      {"capabilities",
+       llvm::json::Object{{"supportsStepInTargetsRequest", false}}}};
   dap.Send(event);
 }
 // "ProcessEvent": {

@DavidSpickett DavidSpickett merged commit 159de36 into llvm:main Jun 4, 2025
10 of 12 checks passed
@da-viper da-viper deleted the fix-explicit-name branch June 4, 2025 09:42
JDevlieghere added a commit to JDevlieghere/llvm-project that referenced this pull request Jun 5, 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.

3 participants