[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