[lldb] [llvm] [llvm][telemetry]Change Telemetry-disabling mechanism. (PR #128534)
    Pavel Labath via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Feb 25 01:14:20 PST 2025
    
    
  
================
@@ -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
    
    
More information about the llvm-commits
mailing list