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