[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