================ @@ -21,6 +21,8 @@ void TelemetryInfo::serialize(Serializer &serializer) const { } Error Manager::dispatch(TelemetryInfo *Entry) { + assert(Config::BuildTimeEnableTelemetry && + "Telemetry should have been enabled"); ---------------- labath wrote:
Won't this be a problem for unit tests? I see you're enabling them, but I suspect they won't actually succeed with this setting turned off.. https://github.com/llvm/llvm-project/pull/128534 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits