[PATCH] D75360: [analyzer][NFC] Tie CheckerRegistry to CheckerManager, allow CheckerManager to be constructed for non-analysis purposes
Artem Dergachev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Mar 15 21:42:41 PDT 2020
NoQ added inline comments.
================
Comment at: clang/include/clang/StaticAnalyzer/Frontend/CheckerRegistry.h:217
+ mgr.template registerChecker<T>();
}
----------------
Szelethus wrote:
> 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.
Maybe put the implementation into the cpp file instead?
================
Comment at: clang/lib/FrontendTool/ExecuteCompilerInvocation.cpp:33
#include "llvm/Support/ErrorHandling.h"
+#include <memory>
using namespace clang;
----------------
Why?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75360/new/
https://reviews.llvm.org/D75360
More information about the cfe-commits
mailing list