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

Aleksei Sidorin a.sidorin at samsung.com
Wed Feb 18 06:32:46 PST 2015


Hello Gábor,

I was thinking that a discussion is dead. It seems like I have fixed all 
the issues that were pointed with this review, but there were no 
replies. So, I was thinking that nobody needs this functionality.
You can try this patch and leave your opinion about it if you have any 
suggestions.
About long command lines... AFAIR command line is a common way to pass 
parameters for all parts of Clang, including CSA. Anyway, there are only 
few checkers that may support this option now so it cannot add a lot to 
the command line currently.


> Hi Aleksei!
>
> What is the state of this patch? Do you still working on it? I am very interested in this feature.
>
> One minor note though: there are some hard limit on some platforms on the length of the command line (as far as I know it's about 8k characters on windows. source: http://support.microsoft.com/kb/830473). Command line also contains include paths and several other valuable information. At Ericsson we can have command lines that are several thousand characters long when invoking the static analyzer. Adding configuration options to the command line might result in hitting the character limit. In the long run I think it would be better to support configuration files.
>
>
> http://reviews.llvm.org/D3967
>
> EMAIL PREFERENCES
>    http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>

-- 
Best regards,
Aleksei Sidorin
Software Engineer,
IMSWL-IMCG, SRR, Samsung Electronics





More information about the cfe-commits mailing list