[PATCH] D152436: [clang][analyzer] Move checker alpha.unix.StdCLibraryFunctions out of alpha.

[Artem Dergachev via Phabricator via cfe-commits]

NoQ added a comment.

It doesn't have to be in `core` just because it's dealing with "core features" of the language.

The `core` package is for checks without which path-sensitive analysis becomes //so incredibly incorrect// that we don't want to support such configuration, we don't want our users to ever use it.

(Ideally `core` checkers shouldn't emit any warnings at all. It has to be possible to disable every particular warning and still have other warnings work correctly. But unfortunately that's not the reality of the situation.)

(I agree `unix` doesn't sound right though. But we already have `MallocChecker` in `unix`, which is arguably way worse. What we really need is, to replace our package system with a hashtag system.)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D152436/new/

https://reviews.llvm.org/D152436