[PATCH] D76545: [clang-tidy] Add a new check group 'experimental-'

[Whisperity via Phabricator via cfe-commits]

whisperity added a comment.

In D76545#1935603 <https://reviews.llvm.org/D76545#1935603>, @aaron.ballman wrote:

> I think we want to keep the experimental checks grouped to their parent module rather than being in a module of their own.


For this, wouldn't the fact that telling Tidy to enable `cppcoreguidelines-*` (i.e. `--checks='-*,cppcoreguidelines-*'`) also enables the `cppcoreguidelines-experimental-*` cause a problem? (This is the original reason why I tinkered in the "top-level" `experimental-` mode.)

In D76545#1935603 <https://reviews.llvm.org/D76545#1935603>, @aaron.ballman wrote:

> I'm not opposed to the idea of experimental checks, but I don't want to see "experimental" become a dumping ground for low-quality checks that we then have to maintain indefinitely.


I think this could (in terms of development, support, etc.) work similarly to the `alpha.` group/package of Clang Static Analyser checkers. I'm not too knowledgeable about CSA - perhaps @Szelethus could help me out, - but their idea of alpha checkers is that they are mature in terms of the general idea, but clunky in terms of working. (The official stance is that anything out of alpha should not crash, anything within alpha might crash, and if you enable it and you crash, you are your own perpetrator.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76545