================
@@ -33,7 +33,15 @@ struct Registry {
 
   struct Snapshot {
     const Decl *EntryPoint;
-    std::vector<bool> BoolStatValues;
+    // Boolean statistics are always set explicitly. If they are not set, their
+    // value is absent resulting in empty CSV cells
+    std::vector<std::optional<bool>> BoolStatValues;
----------------
NagyDonat wrote:

How is this change (where you modify the behavior of the boolean statistics) 
related to the rest of this commit (which handles unsigned statistics)? Does 
this influence anything (I don't see a corresponding change in the test)?

(By the way, does upstream clang have any boolean statistics at all?) 


https://github.com/llvm/llvm-project/pull/162089
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to