[Lldb-commits] [lldb] [LLDB][Telemetry]Define DebuggerTelemetryInfo and related methods (PR #127696)

[Vy Nguyen via lldb-commits]

================
@@ -226,13 +227,45 @@ lldb::SBError SBDebugger::InitializeWithErrorHandling() {
   return error;
 }
 
+#if LLVM_ENABLE_TELEMETRY
+#if ENABLE_BACKTRACES
+static void TelemetryHandler(void *) {
+  // TODO: get the bt into the msg?
+  // Also need to pre-alloc the memory for this entry?
+  lldb_private::telemetry::DebuggerInfo entry;
----------------
oontvoo wrote:

Ok, that's a great point. I'll revert this change and have the crash-handler registered in downstream code (with a new `AtCrash` callback to avoid confusion) Thanks!

https://github.com/llvm/llvm-project/pull/127696