
<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Jun 17, 2013, at 14:08 , Stephen Kelly <<a href="mailto:steveire@gmail.com">steveire@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">Dmitri Gribenko wrote:<br><br><blockquote type="cite">On Mon, Jun 17, 2013 at 11:19 AM, Jordan Rose<br><<a href="mailto:jordan_rose@apple.com">jordan_rose@apple.com</a>> wrote:<br><blockquote type="cite">Hm. Does clang-check really need the static analyzer? Can it be made<br>optional there as well? It seems weird that<br>CLANG_ENABLE_STATIC_ANALYZER=0 would completely prevent clang-check from<br>building.<br></blockquote><br>IIRC, one can do 'clang-check -analyze ...'<br><br>Maybe adding an #ifdef in clang-check source would help.<br><br></blockquote><br>I see no reason to create a version of clang-check which accepts different<span class="Apple-converted-space"> </span><br>arguments based on how it was built. I prefer to simply require the static<span class="Apple-converted-space"> </span><br>analyzer for that tool, and I don't see why it would be a problem.<br></div></blockquote></div><br><div>Well, that's essentially what happens with the core Clang binary and --analyze, though we don't spread that around much.</div><div><br></div><div>I won't block the patch, and we certainly don't want to leave the CMake build broken, but clang-check certainly does more than just 'clang-check -analyze ...', and it seems to be arbitrarily limiting to remove that. I guess I would include a FIXME to properly conditionalize clang-check for CLANG_ENABLE_STATIC_ANALYZER, but then go ahead and commit.</div><div><br></div><div>Jordan</div></body></html>