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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
