[PATCH] [Proposal] [Analyzer] Individual options for checkers

Ted Kremenek kremenek at apple.com
Tue Aug 19 14:53:28 PDT 2014


Thanks Aleksei.  That explanation helps quite a bit.  I've responded in Phabricator with some comments.

> On Aug 18, 2014, at 12:43 AM, Aleksei Sidorin <a.sidorin at samsung.com> wrote:
> 
> Hello Ted,
> 
> This patch (its current version) implements a helper function to retrieve individual options for checkers. Currently the only way to set options for checkers is a registration of a special checker. I tried to implement a more convenient way to do this. For example,
> ```
> checker->setCheckEnabled(Expr::SET_VolatileRead, getBooleanOption("EmitOnVolatileRead", true, checker));
> ```
> instead of registration of a new checker (Malloc and CString checkers have even macros to do this). It is only a small convenience and doesn't break any existing functionality.
> This should also work for groups of checkers. For example, `... -analyzer-config unix=some_option` may be specified to set behaviour for all checkers in `unix` group.
> 
> http://reviews.llvm.org/D3967
> 
> 




More information about the cfe-commits mailing list