Fix build with CLANG_ENABLE_STATIC_ANALYZER
Stephen Kelly
steveire at gmail.com
Mon Jun 17 14:08:59 PDT 2013
Dmitri Gribenko wrote:
> On Mon, Jun 17, 2013 at 11:19 AM, Jordan Rose
> <jordan_rose at apple.com> wrote:
>> Hm. Does clang-check really need the static analyzer? Can it be made
>> optional there as well? It seems weird that
>> CLANG_ENABLE_STATIC_ANALYZER=0 would completely prevent clang-check from
>> building.
>
> IIRC, one can do 'clang-check -analyze ...'
>
> Maybe adding an #ifdef in clang-check source would help.
>
I see no reason to create a version of clang-check which accepts different
arguments based on how it was built. I prefer to simply require the static
analyzer for that tool, and I don't see why it would be a problem.
Thanks,
Steve.
More information about the cfe-commits
mailing list