HACK: Smuggle in the checker name into the CheckerFamily class

Author: steakhal

clang/include/clang/StaticAnalyzer/Core/Checker.h

@@ -526,7 +526,7 @@ class CheckerBase : public CheckerFrontend, public CheckerBackend {
 public:
   /// Attached to nodes created by this checker class when the ExplodedGraph is
   /// dumped for debugging.
-  StringRef getDebugName() const override;
+  StringRef getDebugName() const final;
 };
 
 /// Simple checker classes that implement one frontend (i.e. checker name)
@@ -545,16 +545,22 @@ class Checker : public CheckerBase, public CHECKs... {
 /// Checker families (where a single backend class implements multiple related
 /// frontends) should derive from this template, specify all the implemented
 /// callbacks (i.e. classes like `check::PreStmt` or `eval::Call`) as template
-/// arguments of `FamilyChecker` and implement the pure virtual method
-/// `StringRef getDebugName()` which is inherited from `ProgramPointTag`
-/// and should return a string identifying the class for debugging purposes.
+/// arguments of `FamilyChecker`.
 template <typename... CHECKs>
 class CheckerFamily : public CheckerBackend, public CHECKs... {
 public:
   template <typename CHECKER>
   static void _register(CHECKER *Chk, CheckerManager &Mgr) {
+    Chk->CheckerBackendName = Mgr.getCurrentCheckerName();
     (CHECKs::_register(Chk, Mgr), ...);
   }
+
+  /// Attached to nodes created by this checker class when the ExplodedGraph is
+  /// dumped for debugging.
+  StringRef getDebugName() const final { return CheckerBackendName; }
+
+private:
+  CheckerNameRef CheckerBackendName;
 };
 
 template <typename EVENT>

clang/lib/StaticAnalyzer/Checkers/DivZeroChecker.cpp

@@ -39,9 +39,6 @@ class DivZeroChecker : public CheckerFamily<check::PreStmt<BinaryOperator>> {
                                                categories::TaintedData};
 
   void checkPreStmt(const BinaryOperator *B, CheckerContext &C) const;
-
-  /// Identifies this checker family for debugging purposes.
-  StringRef getDebugName() const override { return "DivZeroChecker"; }
 };
 } // end anonymous namespace
 

clang/lib/StaticAnalyzer/Checkers/VirtualCallChecker.cpp

@@ -54,9 +54,6 @@ class VirtualCallChecker
   void checkEndFunction(const ReturnStmt *RS, CheckerContext &C) const;
   void checkPreCall(const CallEvent &Call, CheckerContext &C) const;
 
-  /// Identifies this checker family for debugging purposes.
-  StringRef getDebugName() const override { return "VirtualCallChecker"; }
-
 private:
   void registerCtorDtorCallInState(bool IsBeginFunction,
                                    CheckerContext &C) const;

clang/test/Analysis/ftime-trace.cpp

@@ -39,7 +39,7 @@
 
 // Finally, each checker call back is also present:
 //
-// CHECK:          "name": "Total Stmt:DivZeroChecker",
+// CHECK:          "name": "Total Stmt:core.DivideZero",
 // CHECK-NEXT:     "args": {
 // CHECK-NEXT:         "count": {{[0-9]+}},
 // CHECK-NEXT:         "avg ms": {{[0-9]+}}