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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to