<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On May 8, 2013, at 3:52 PM, Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com">gribozavr@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="letter-spacing: 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;">On Sat, May 4, 2013 at 3:47 AM, Fariborz Jahanian <<a href="mailto:fjahanian@apple.com">fjahanian@apple.com</a>> wrote:<br><blockquote type="cite">Author: fjahanian<br>Date: Fri May 3 19:47:28 2013<br>New Revision: 181081<br><br>URL:<span class="Apple-converted-space"> </span><a href="http://llvm.org/viewvc/llvm-project?rev=181081&view=rev">http://llvm.org/viewvc/llvm-project?rev=181081&view=rev</a><br>Log:<br>[doc parsing]: Make warning about unknown command<br>tags off by default for now. Move diagnostic code<br>to DiagnosticCommentKinds.td. //<span class="Apple-converted-space"> </span><a href="rdar://12381408">rdar://12381408</a><br></blockquote><br>I think this should be moved a separate group (for example, a separate<br>flag under DocumentationPedantic group). It causes way too much noise<br>even for LLVM and Clang.<br><br><a href="http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-debian-fast/builds/2344/steps/build/logs/stdio">http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-debian-fast/builds/2344/steps/build/logs/stdio</a><br></div></blockquote><div><br></div>I agree. I was just looking at the log.</div><div><br><blockquote type="cite"><div style="letter-spacing: 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;"><br>(There's also a crash somewhere in diagnostic printing, BTW…)<br></div></blockquote><div><br></div>I just fixed the crash<br><blockquote type="cite"><div style="letter-spacing: 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;"><br>This change should be also committed to 3.3.<br><br>Doug, Fariborz, what do you think about this proposed change?<br><br>Dmitri<br><br>--<br>main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if<br>(j){printf("%d\n",i);}}} /*Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com">gribozavr@gmail.com</a>>*/</div></blockquote></div><br></body></html>