https://github.com/DrSergei created 
https://github.com/llvm/llvm-project/pull/200133

Without this patch event listener registration was skipped, as a result 
`Modules` view in UI was not displayed in case of launching target via 
`launchCommands`.

>From 3f3bfbe4c83eda930207684d801409a34ccd310b Mon Sep 17 00:00:00 2001
From: Sergei Druzhkov <[email protected]>
Date: Thu, 28 May 2026 11:30:07 +0300
Subject: [PATCH] [lldb-dap] Use SetTarget for launch commands

---
 .../launch/TestDAP_launch_extra_launch_commands.py       | 9 ++++++++-
 lldb/tools/lldb-dap/Handler/RequestHandler.cpp           | 2 +-
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git 
a/lldb/test/API/tools/lldb-dap/launch/TestDAP_launch_extra_launch_commands.py 
b/lldb/test/API/tools/lldb-dap/launch/TestDAP_launch_extra_launch_commands.py
index 46a1b0a6165c9..ad48af6364aba 100644
--- 
a/lldb/test/API/tools/lldb-dap/launch/TestDAP_launch_extra_launch_commands.py
+++ 
b/lldb/test/API/tools/lldb-dap/launch/TestDAP_launch_extra_launch_commands.py
@@ -57,8 +57,15 @@ def test(self):
         # After execution, program should launch
         self.verify_commands("launchCommands", output, launchCommands)
         self.verify_commands("postRunCommands", output, postRunCommands)
+
+        # Finish configuration and continue target
+        self.verify_configuration_done()
+
+        # Check that we got module events from target
+        modules = self.dap_server.wait_for_module_events()
+        self.assertGreater(len(modules), 0)
+
         # Verify the "stopCommands" here
-        self.continue_to_next_stop()
         output = self.get_console()
         self.verify_commands("stopCommands", output, stopCommands)
 
diff --git a/lldb/tools/lldb-dap/Handler/RequestHandler.cpp 
b/lldb/tools/lldb-dap/Handler/RequestHandler.cpp
index e44e09e4741e4..f375a409d251a 100644
--- a/lldb/tools/lldb-dap/Handler/RequestHandler.cpp
+++ b/lldb/tools/lldb-dap/Handler/RequestHandler.cpp
@@ -261,7 +261,7 @@ llvm::Error BaseRequestHandler::LaunchProcess(
 
       // The custom commands might have created a new target so we should use
       // the selected target after these commands are run.
-      dap.target = dap.debugger.GetSelectedTarget();
+      dap.SetTarget(dap.debugger.GetSelectedTarget());
     }
   }
 

_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to