[PATCH] D75271: [analyzer][NFC] Change LangOptions to CheckerManager in the shouldRegister* functions.
Balogh, Ádám via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Mar 5 10:23:00 PST 2020
baloghadamsoftware requested changes to this revision.
baloghadamsoftware added a comment.
This revision now requires changes to proceed.
This patch still does not help. Most of the functions of `CheckerManager` are non-const thus they cannot be called on a const object. E.g. `getAnalyzerOptions()` is non-const (why?). For error reporting `getASTContext()` should be const as well (to access the diagnostics engine). Thus either `CheckerManager` should be changed by setting its getters to const or the `shouldRegister`//XXX//`()` should take a non-const reference to `CheckerManager`. I would prefer the former one.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75271/new/
https://reviews.llvm.org/D75271
More information about the cfe-commits
mailing list