================
@@ -7944,9 +7944,13 @@ void ASTReader::PrintStats() {
   std::fprintf(stderr, "*** AST File Statistics:\n");
 
   unsigned NumTypesLoaded =
-      TypesLoaded.size() - llvm::count(TypesLoaded, QualType());
+      TypesLoaded.size() - std::count(TypesLoaded.materialisedBegin(),
----------------
dwblaikie wrote:

This seems confusing - I'd have thought any materialised type was a loaded 
type? So `NumTypesLoaded = std::count(TypesLoaded.materialisedBegin(), 
TypesLoaded.materialisedEnd())`?

(similarly for `DeclsLoaded` below)

Like wouldn't this result in `NumTypesLoaded` too high - for unmaterialized 
values they're all assumed to be loaded? Which sounds backwards.

https://github.com/llvm/llvm-project/pull/66430
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to