oontvoo wrote: @labath > IOW, can we drop the exit_desc field from this struct?
Well, we need a way to distinguish the start and the exit entries How about just keeping `exit_code` (which is set to zero) > I still think it's important to have a simple and succinct way to check > whether telemetry is "actually" enabled at a given moment We can have a `static TelemetryManager::TelemetryCurrentEnabled()` which does the check. https://github.com/llvm/llvm-project/pull/127696 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits