[PATCH] D79420: [analyzer] Make NonNullParamChecker as dependency for StdCLibraryFunctionsChecker
Kristóf Umann via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue May 5 09:08:43 PDT 2020
Szelethus accepted this revision.
Szelethus added a comment.
This revision is now accepted and ready to land.
I'm a bit unsure about the naming, because it's not technically true that `NonNullParamChecker` is a dependency of `StdCLibraryFunctionsChecker`. The actual relationship is different, because we only want to change the evaluation order, but then again, you can argue that this is in fact a dependency. So its like whatever, LGTM :^). Could you just drop a `// NOTE:` though about this?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79420/new/
https://reviews.llvm.org/D79420
More information about the cfe-commits
mailing list