[PATCH] D84600: [Analyzer] Support note tags for smart ptr checker
Nithin VR via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Aug 8 10:27:53 PDT 2020
vrnithinkumar added a comment.
In D84600#2203604 <https://reviews.llvm.org/D84600#2203604>, @Szelethus wrote:
> Layering violations are a running theme in the analyzer -- CheckerRegistry and the entire MallocChecker fiasco <https://www.youtube.com/watch?v=eqKeqHRAhQM> are two glaring examples. Luckily, this isn't a severe case so I wouldn't worry about it much.
>
> I've been following your patches, but them mentors answer way ahead of me and are very thorough :^) Best of luck with the finale!
Thanks for sharing checker dependency talk.
Also fo suggesting to separate checker and modeling from beginning. :)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D84600/new/
https://reviews.llvm.org/D84600
More information about the cfe-commits
mailing list