[clang] [LifetimeSafety] Add missing origins stats for lifetime analysis (PR #166568)

Utkarsh Saxena via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 26 01:08:46 PST 2025


================
@@ -16,7 +16,16 @@
 
 #include "clang/AST/Decl.h"
 #include "clang/AST/Expr.h"
+#include "clang/AST/RecursiveASTVisitor.h"
+#include "clang/AST/TypeBase.h"
 #include "clang/Analysis/Analyses/LifetimeSafety/Utils.h"
+#include "llvm/ADT/StringMap.h"
+#include "llvm/Support/raw_ostream.h"
+
+namespace clang::lifetimes {
+
+struct LifetimeSafetyStats;
----------------
usx95 wrote:

Ah sorry for missing this in my original suggestion. I think you have added this because it creates a cyclic header includes if we include LifetimeSafety.h here

Maybe have a separate header for Stats "clang/include/clang/Analysis/Analyses/LifetimeSafety/Stats.h" `printStats`​ can live in clang/lib/Analysis/LifetimeSafety/Stats.cpp
`collectLifetimeStats`​ would need to still live in LifetimeSafety.cpp

https://github.com/llvm/llvm-project/pull/166568


More information about the cfe-commits mailing list