Minor refactoring of the static functions

Author: Debadri Basak

clang/include/clang/Analysis/Analyses/LifetimeSafety/LifetimeSafety.h

@@ -80,19 +80,9 @@ class LifetimeSafetyAnalysis {
   LiveOriginsAnalysis &getLiveOrigins() const { return *LiveOrigins; }
   FactManager &getFactManager() { return FactMgr; }
 
-  static void PrintStats(llvm::raw_ostream &OS) {
-    llvm::errs() << "\n*** LifetimeSafety Missing Origin Stats "
-                    "(expression_type : count) :\n";
-    for (const auto &[expr, count] : LifetimeSafetyAnalysis::count) {
-      OS << expr << " : " << count << '\n';
-    }
-  }
+  static void PrintStats(llvm::raw_ostream &OS);
 
-  static void UpdateMissingOriginCount(const OriginManager &OM) {
-    for (const auto &[expr, missing_origin_count] : OM.getMissingOrigins()) {
-      LifetimeSafetyAnalysis::count[std::string(expr)] += missing_origin_count;
-    }
-  }
+  static void UpdateMissingOriginCount(const OriginManager &OM);
 
 private:
   AnalysisDeclContext &AC;

clang/lib/Analysis/LifetimeSafety/LifetimeSafety.cpp

@@ -39,6 +39,19 @@ LifetimeSafetyAnalysis::LifetimeSafetyAnalysis(AnalysisDeclContext &AC,
                                                LifetimeSafetyReporter *Reporter)
     : AC(AC), Reporter(Reporter) {}
 
+void LifetimeSafetyAnalysis::PrintStats(llvm::raw_ostream &OS) {
+    llvm::errs() << "\n*** LifetimeSafety Missing Origin Stats "
+                    "(expression_type : count) :\n";
+    for (const auto &[expr, count] : LifetimeSafetyAnalysis::MissingOriginCount) {
+      OS << expr << " : " << count << '\n';
+    }
+  }
+
+void LifetimeSafetyAnalysis::UpdateMissingOriginCount(const OriginManager &OM) {
+    for (const auto &[expr, missing_origin_count] : OM.getMissingOrigins()) {
+      LifetimeSafetyAnalysis::MissingOriginCount[std::string(expr)] += missing_origin_count;
+    }
+  }
 void LifetimeSafetyAnalysis::run() {
   llvm::TimeTraceScope TimeProfile("LifetimeSafetyAnalysis");