[PATCH] D69731: [analyzer] CheckerContext: Make the Preprocessor available
Csaba Dabis via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Nov 1 19:28:01 PDT 2019
Charusso added a comment.
In D69731#1730956 <https://reviews.llvm.org/D69731#1730956>, @Charusso wrote:
> I am thinking of a callback which is something like:
>
> void checkBeginAnalysis(const Decl *D, BugReporter &BR) const;
>
>
> so it would be easy and meaningful to have a place for the `Preprocessor` logic. Do you think it would worth it?
Well, it made me mad to create setters / public fields and to rely on only the `AnalysisManager`. I am going to publish my first real checker, and it turns out being cool: D69745 <https://reviews.llvm.org/D69745>
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69731/new/
https://reviews.llvm.org/D69731
More information about the cfe-commits
mailing list