remove unused

Author: oontvoo

lldb/include/lldb/Core/Telemetry.h

@@ -106,6 +106,7 @@ struct TargetInfo : public LLDBBaseTelemetryInfo {
     return (T->getKind() & LLDBEntryKind::TargetInfo) ==
            LLDBEntryKind::TargetInfo;
   }
+  void serialize(llvm::telemetry::Serializer &serializer) const override;
 };
 
 struct DebuggerInfo : public LLDBBaseTelemetryInfo {
@@ -139,17 +140,6 @@ class TelemetryManager : public llvm::telemetry::Manager {
 
   virtual llvm::StringRef GetInstanceName() const = 0;
 
-  /// The following methods are for reporting the load of an executable.
-  /// One is invoked at the beginning of the process and the other at
-  /// the end.
-  /// This is done in two passes to avoid losing date in case of any error/crash
-  /// during the action.
-  ///
-  /// Invoked at the begining of the load of the main-executable.
-  virtual void AtMainExecutableLoadStart(TargetInfo *entry);
-  /// Invoked at the end of the load.
-  virtual void AtMainExecutableLoadEnd(TargetInfo *entry);
-
   static TelemetryManager *GetInstance();
 
 protected:

lldb/source/Core/Telemetry.cpp

@@ -68,14 +68,6 @@ void DebuggerInfo::serialize(Serializer &serializer) const {
   serializer.write("is_exit_entry", is_exit_entry);
 }
 
-void LLDBBaseTelemetryInfo::serialize(Serializer &serializer) const {
-  serializer.write("entry_kind", getKind());
-  serializer.write("session_id", SessionId);
-  serializer.write("start_time", ToNanosec(start_time));
-  if (end_time.has_value())
-    serializer.write("end_time", ToNanosec(end_time.value()));
-}
-
 void TargetInfo::serialize(Serializer &serializer) const {
   LLDBBaseTelemetryInfo::serialize(serializer);
 
@@ -88,14 +80,6 @@ void TargetInfo::serialize(Serializer &serializer) const {
   }
 }
 
-void MiscTelemetryInfo::serialize(Serializer &serializer) const {
-  LLDBBaseTelemetryInfo::serialize(serializer);
-  serializer.write("target_uuid", target_uuid);
-  serializer.beginObject("meta_data");
-  for (const auto &kv : meta_data)
-    serializer.write(kv.first, kv.second);
-  serializer.endObject();
-}
 
 TelemetryManager::TelemetryManager(std::unique_ptr<Config> config)
     : m_config(std::move(config)), m_id(MakeUUID()) {}
@@ -119,8 +103,8 @@ class NoOpTelemetryManager : public TelemetryManager {
   }
 
   explicit NoOpTelemetryManager()
-      : TelemetryManager(std::make_unique<LLDBConfig>(
-            /*EnableTelemetry*/ false, /*DetailedCommand*/ false)) {}
+      : TelemetryManager(std::make_unique<::llvm::telemetry::Config>(
+            /*EnableTelemetry*/ false)) {}
 
   virtual llvm::StringRef GetInstanceName() const override {
     return "NoOpTelemetryManager";

lldb/source/Target/Process.cpp

@@ -1066,7 +1066,7 @@ bool Process::SetExitStatus(int status, llvm::StringRef exit_string) {
   // Use a mutex to protect setting the exit status.
   std::lock_guard<std::mutex> guard(m_exit_status_mutex);
   Debugger *debugger = &(GetTarget().GetDebugger());
-  telemetry::ScopedDispatch<telemetry::TargetInfo> helper(debugger);
+  telemetry::ScopedDispatcher<telemetry::TargetInfo> helper(debugger);
   // Save the Module UUID since the Module might be gone by end of scope.
   std::string target_uuid =
       GetTarget().GetExecutableModule()->GetUUID().GetAsString();