<div style="font-family: arial, helvetica, sans-serif; font-size: 10pt"><div dir="ltr"><div class="gmail_default" style>FWIW, I'm not excited about 5% build time reduction for this warning.</div><div class="gmail_default" style>
<br>I'd like to have it on, but I'd like someone to spend some time with a profiler and see if the overhead can be pushed down closer to 1 or 2%... =/ Sadly, I don't have a lot of time to stare at this part of compile-time performance.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Dec 26, 2012 at 10:10 AM, Douglas Gregor <span dir="ltr"><<a href="mailto:dgregor@apple.com" target="_blank">dgregor@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div>Yeah, LGTM. <br><br>Sent from my iPhone</div><div><div class="h5"><div><br>On Dec 24, 2012, at 6:35 PM, Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:<br>
<br></div><blockquote type="cite"><div><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" target="_blank">gribozavr@gmail.com</a>>:<br>
<div><div>> 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" target="_blank">gribozavr@gmail.com</a>>*/<br>
</div></div></blockquote></div><br></div>
</div></blockquote></div></div></div>
</blockquote></div><br></div></div>