Format

Author: qxy11

lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp

@@ -1025,9 +1025,9 @@ Status ProcessGDBRemote::HandleConnectionRequest(const GPUActions &gpu_action) {
                                  process_sp->GetTarget().shared_from_this());
   LLDB_LOG(log, "ProcessGDBRemote::HandleConnectionRequest(): successfully "
                 "created process!!!");
-  auto event_sp = std::make_shared<Event>(
-      Target::eBroadcastBitNewTargetSpawned,
-      new Target::TargetEventData(gpu_target_sp));
+  auto event_sp =
+      std::make_shared<Event>(Target::eBroadcastBitNewTargetSpawned,
+                              new Target::TargetEventData(gpu_target_sp));
   GetTarget().BroadcastEvent(event_sp);
   return Status();
 }

lldb/tools/lldb-dap/DAP.cpp

@@ -1624,15 +1624,19 @@ void DAP::EventThread() {
               "startDebugging", std::move(start_debugging_args));
         }
       } else if (lldb::SBBreakpoint::EventIsBreakpointEvent(event)) {
-        lldb::SBBreakpoint bp = lldb::SBBreakpoint::GetBreakpointFromEvent(event);
-        if (!bp.IsValid()) continue;
-        
+        lldb::SBBreakpoint bp =
+            lldb::SBBreakpoint::GetBreakpointFromEvent(event);
+        if (!bp.IsValid())
+          continue;
+
         lldb::SBTarget event_target = bp.GetTarget();
-        
+
         // Find the DAP instance that owns this target
-        DAP* dap_instance = DAPSessionManager::GetInstance().FindDAPForTarget(event_target);
-        if (!dap_instance) continue;
-        
+        DAP *dap_instance =
+            DAPSessionManager::GetInstance().FindDAPForTarget(event_target);
+        if (!dap_instance)
+          continue;
+
         if (event_mask & lldb::SBTarget::eBroadcastBitBreakpointChanged) {
           auto event_type =
               lldb::SBBreakpoint::GetBreakpointEventTypeFromEvent(event);
@@ -1666,18 +1670,22 @@ void DAP::EventThread() {
       } else if (event_mask & lldb::eBroadcastBitError ||
                  event_mask & lldb::eBroadcastBitWarning) {
         // Global debugger events - send to all DAP instances
-        std::vector<DAP*> active_instances = DAPSessionManager::GetInstance().GetActiveSessions();
-        for (DAP* dap_instance : active_instances) {
-          if (!dap_instance) continue;
-          
+        std::vector<DAP *> active_instances =
+            DAPSessionManager::GetInstance().GetActiveSessions();
+        for (DAP *dap_instance : active_instances) {
+          if (!dap_instance)
+            continue;
+
           lldb::SBStructuredData data =
               lldb::SBDebugger::GetDiagnosticFromEvent(event);
-          if (!data.IsValid()) continue;
-          
+          if (!data.IsValid())
+            continue;
+
           std::string type = GetStringValue(data.GetValueForKey("type"));
           std::string message = GetStringValue(data.GetValueForKey("message"));
-          dap_instance->SendOutput(OutputType::Important,
-                                  llvm::formatv("{0}: {1}", type, message).str());
+          dap_instance->SendOutput(
+              OutputType::Important,
+              llvm::formatv("{0}: {1}", type, message).str());
         }
       } else if (event.BroadcasterMatchesRef(broadcaster)) {
         if (event_mask & eBroadcastBitStopEventThread) {

lldb/tools/lldb-dap/DAP.h

@@ -69,16 +69,16 @@ using ClientFeature = protocol::ClientFeature;
 class DAPSessionManager {
 public:
   /// Get the singleton instance of the DAP session manager
-  static DAPSessionManager& GetInstance();
+  static DAPSessionManager &GetInstance();
 
   /// Register a DAP session
-  void RegisterSession(lldb::IOObjectSP io, DAP* dap);
+  void RegisterSession(lldb::IOObjectSP io, DAP *dap);
 
   /// Unregister a DAP session
   void UnregisterSession(lldb::IOObjectSP io);
 
   /// Get all active DAP sessions
-  std::vector<DAP*> GetActiveSessions();
+  std::vector<DAP *> GetActiveSessions();
 
   /// Disconnect all active sessions
   void DisconnectAllSessions();
@@ -93,10 +93,11 @@ class DAPSessionManager {
   std::optional<lldb::SBDebugger> GetSharedDebugger();
 
   /// Get or create event thread for a specific debugger
-  std::shared_ptr<std::thread> GetEventThreadForDebugger(lldb::SBDebugger debugger, DAP* requesting_dap);
+  std::shared_ptr<std::thread>
+  GetEventThreadForDebugger(lldb::SBDebugger debugger, DAP *requesting_dap);
 
   /// Find the DAP instance that owns the given target
-  DAP* FindDAPForTarget(lldb::SBTarget target);
+  DAP *FindDAPForTarget(lldb::SBTarget target);
 
   /// Clean up shared resources when the last session exits
   void CleanupSharedResources();
@@ -106,22 +107,23 @@ class DAPSessionManager {
   ~DAPSessionManager() = default;
 
   // Non-copyable and non-movable
-  DAPSessionManager(const DAPSessionManager&) = delete;
-  DAPSessionManager& operator=(const DAPSessionManager&) = delete;
-  DAPSessionManager(DAPSessionManager&&) = delete;
-  DAPSessionManager& operator=(DAPSessionManager&&) = delete;
+  DAPSessionManager(const DAPSessionManager &) = delete;
+  DAPSessionManager &operator=(const DAPSessionManager &) = delete;
+  DAPSessionManager(DAPSessionManager &&) = delete;
+  DAPSessionManager &operator=(DAPSessionManager &&) = delete;
 
   std::mutex sessions_mutex_;
   std::condition_variable sessions_condition_;
-  std::map<lldb::IOObjectSP, DAP*> active_sessions_;
-  
+  std::map<lldb::IOObjectSP, DAP *> active_sessions_;
+
   /// Optional shared debugger instance set when the native process
   /// spawns a new GPU target
   std::optional<lldb::SBDebugger> shared_debugger_;
-  
+
   /// Map from debugger ID to its event thread used for when
   /// multiple DAP sessions are using the same debugger instance.
-  std::map<lldb::user_id_t, std::shared_ptr<std::thread>> debugger_event_threads_;
+  std::map<lldb::user_id_t, std::shared_ptr<std::thread>>
+      debugger_event_threads_;
 };
 
 enum class OutputType { Console, Important, Stdout, Stderr, Telemetry };
@@ -138,7 +140,7 @@ enum class ReplMode { Variable = 0, Command, Auto };
 
 struct DAP {
   friend class DAPSessionManager;
-  
+
   /// Path to the lldb-dap binary itself.
   static llvm::StringRef debug_adapter_path;
 

lldb/tools/lldb-dap/Handler/AttachRequestHandler.cpp

@@ -29,7 +29,8 @@ namespace lldb_dap {
 /// Since attaching is debugger/runtime specific, the arguments for this request
 /// are not part of this specification.
 Error AttachRequestHandler::Run(const AttachRequestArguments &args) const {
-  // Initialize DAP debugger and related components if not sharing previously launched debugger.
+  // Initialize DAP debugger and related components if not sharing previously
+  // launched debugger.
   bool use_shared_debugger = args.targetIdx != UINT32_MAX;
   if (Error err = dap.InitializeDebugger(use_shared_debugger)) {
     return err;

lldb/tools/lldb-dap/Handler/LaunchRequestHandler.cpp

@@ -22,7 +22,7 @@ namespace lldb_dap {
 
 /// Launch request; value of command field is 'launch'.
 Error LaunchRequestHandler::Run(const LaunchRequestArguments &arguments) const {
-  // Initialize DAP debugger 
+  // Initialize DAP debugger
   if (Error err = dap.InitializeDebugger(false /* use_shared_debugger */)) {
     return err;
   }