<div dir="ltr">Other than Takumi's comments the patches themselves look fine. No objections here.<div><br></div><div>-eric</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Dec 24, 2012 at 2:07 PM, NAKAMURA Takumi <span dir="ltr"><<a href="mailto:geek4civic@gmail.com" target="_blank">geek4civic@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">IMO, it would be odd with -Wno-documentation-deprecated-sync.<br>
Could we clean them up?<br>
<br>
...Takumi<br>
<br>
2012/12/25 Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com">gribozavr@gmail.com</a>>:<br>
<div class="HOEnZb"><div class="h5">> Hello,<br>
><br>
> These patches enable -Wdocumentation<br>
> -Wno-documentation-deprecated-sync for LLVM and Clang.  Please review.<br>
><br>
> Nakamura-san has cleaned up the last documentation warnings, so<br>
> enabling these flags now does not introduce new warnings during build.<br>
><br>
> Build time difference is 4.4% .. 5.8% (95% confidence interval).<br>
><br>
> Measurements are for configure-based Release+Asserts<br>
> LLVM+Clang+compiler-rt build on a quad-core Linux desktop with SSD.<br>
><br>
> Dmitri<br>
><br>
> === Raw data and calculations ===<br>
><br>
> Normal build:<br>
> 12m24s = 744 s<br>
> 12m16s = 736 s<br>
> 12m25s = 745 s<br>
><br>
> mean: 741.8 s<br>
> std dev: 5.0<br>
> 95% confidence interval: 736.1 s to 747.5 s<br>
><br>
> With -Wdocumentation:<br>
> 12m58s = 778s<br>
> 12m59s = 779<br>
> 13m0s  = 780s<br>
><br>
> mean: 779.0 s<br>
> std dev: 1<br>
> 95% confidence interval: 779.1 s to 780.1 s<br>
><br>
> Single core build time difference: 130.4 s .. 172.0 s<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>>*/<br>
</div></div></blockquote></div><br></div>