[PATCH] D152436: [clang][analyzer] Move checker alpha.unix.StdCLibraryFunctions out of alpha.
Balázs Benics via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 11 09:13:38 PDT 2023
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
More information about the cfe-commits
mailing list