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
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
