r179634 - [analyzer] Add experimental option "leak-diagnostics-reference-allocation".

Jordan Rose jordan_rose at apple.com
Wed Apr 17 09:08:44 PDT 2013

On Apr 16, 2013, at 14:44 , Ted Kremenek <kremenek at apple.com> wrote:

> +/// the allocatin site (where possible).

Typo here.

Separating analyzer config options out into different files does make sense. The reason I had hesitated to do it was because of the cached options (Optional<Bool>), but several of those could be redesigned in the same way as what you've done for RetainCountChecker, moving them to be local instance variables. A finite number of string parses for each ExprEngine isn't so bad.

No immediate call to action here, but it would be nice to have a plan to stop bloating AnalyzerOptions the way I have been.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130417/88962b6d/attachment.html>

More information about the cfe-commits mailing list