[PATCH] D36067: [analyzer] Create infrastructure for organizing and declaring analyzer configs.

Gábor Horváth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 31 03:13:03 PDT 2017


xazax.hun added a comment.

I like the directions of this patch.
In general, I am in favor of explicitly registering the options from user defined checkers. 
But changing a config option will now break the command line compatibility, so I wonder how do we want to handle this:

- Have a list of no-op configs that we accept but warn that it has been replaced/removed?
- Just do not care and break compatibility?
- Something else?

I have a wishlist for this feature but I am perfectly fine to only address those in follow-up patches:

- Ability to set descriptions and maybe default values in the Checkers.td
- Command line argument to dump the list of options with descriptions (and defaults)


https://reviews.llvm.org/D36067





More information about the cfe-commits mailing list