[PATCH] D158156: [analyzer] Add C++ array delete checker
Balázs Kéri via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 21 07:15:03 PDT 2023
balazske added a comment.
I found that this concept of "fake" checkers (that change only options of a common checker instance) has more problems (checker dependencies, callback order, separation of "modeling" and bug report generation independently), it is better to avoid this. It is really the same as having options of a single checker (separate checkers are not the same as options of a checker), otherwise a common base class or common code module is a better option.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D158156/new/
https://reviews.llvm.org/D158156
More information about the cfe-commits
mailing list