steakhal accepted this revision. steakhal added a comment. This revision is now accepted and ready to land.
As I don't use this checker, thus I cannot speak of my experience. However, the reasons look solid and I'm fine with moving this checker to `unix.StdCLibraryFunctions`. Let some time for other reviewers to object before landing this. Lets say, one week from now. @xazax.hun @NoQ? As next steps, I'd be glad set the default value of `ModelPOSIX` to `true`. I don't see much harm doing so. (And maybe getting rid of that checker option entirely.) ================ Comment at: clang/docs/analyzer/checkers.rst:979-982 +.. _unix-StdCLibraryFunctions: + +unix.StdCLibraryFunctions (C) +""""""""""""""""""""""""""""""""""" ---------------- Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D152436/new/ https://reviews.llvm.org/D152436 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
