Szelethus marked 2 inline comments as done.
Szelethus added inline comments.


================
Comment at: clang/include/clang/StaticAnalyzer/Frontend/CheckerRegistry.h:217
+    mgr.template registerChecker<T>();
   }
 
----------------
baloghadamsoftware wrote:
> Why do we need `MGR` here as template parameter? Is this function also used 
> for other class instances than `CheckerManager`? I fail to see such 
> invocation.
Include cycles :) `CheckerManager.h` includes `CheckerRegistry.h`, so we can 
only forward declare `CheckerManager`, hence the need for a template parameter.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D75360/new/

https://reviews.llvm.org/D75360



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to