================ @@ -59,11 +60,21 @@ struct LLDBBaseTelemetryInfo : public llvm::telemetry::TelemetryInfo { /// The base Telemetry manager instance in LLDB /// This class declares additional instrumentation points /// applicable to LLDB. -class TelemetryManager : public llvm::telemetry::Manager { +class TelemetryManager : public llvm::telemetry::Manager, + public PluginInterface { public: + llvm::Error preDispatch(llvm::telemetry::TelemetryInfo *entry) override; + + // Plugin interface + llvm::StringRef GetPluginName() override { return "TelemetryManager"; } ---------------- labath wrote:
Can we delete this (due to not needing to instantiate the base class)? https://github.com/llvm/llvm-project/pull/126588 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits