<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 20, 2016 at 3:14 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.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="ltr"><br><br><div class="gmail_quote"><span class=""><div dir="ltr">On Wed, Jan 20, 2016 at 3:12 PM Sean Silva <<a href="mailto:chisophugis@gmail.com" target="_blank">chisophugis@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">silvas added a comment.<br>
<br>
In <a href="http://reviews.llvm.org/D15999#331601" rel="noreferrer" target="_blank">http://reviews.llvm.org/D15999#331601</a>, @kromanova wrote:<br>
<br>
> In <a href="http://reviews.llvm.org/D15999#330794" rel="noreferrer" target="_blank">http://reviews.llvm.org/D15999#330794</a>, @silvas wrote:<br>
><br>
> > This may sound stupid, but: can you benchmark the time it takes to build some project (that actually uses intrinsics in most translation units, e.g. a game) with the headers w/ and w/o the doxygen comments to check that all the extra comment skipping doesn't affect compilation time? I.e. run your script to add the comments for "all" the intrinsic headers (similar to what you expect the final state to be after all these patches) and test the build time of a game (and compare with the unmodified headers).<br>
> ><br>
> > Also, can you post a patch that changes "all" the headers to have doxygen comments like you intend, so that others can test and verify?<br>
><br>
><br>
> Out of curiosity, do you know if the impact to the build time for Eric's change of starting to use the target attributes instead of conditional inclusion was measured (r239883) on a large scale application. If so, what were the results? I should probably include Eric.<br>
<br>
<br>
I don't think we did any testing at SCE, but we probably should have. I don't think Google's primary codebases (nor Apple's, or anybody else pretty much) include the intrinsic headers in basically every TU so the impact is probably not a huge concern overall. However, games include these headers in basically every TU so we at SCE should take a look.<br>
<br></blockquote><div><br></div></span><div>This seems to be a pretty broad brush to paint things with and an assumption I wouldn't be willing to make.</div></div></div></blockquote><div><br></div><div>Every vendor obviously has some projects that end up pulling in intrinsics in basically every TU. It's just that for SCE those are the rule and not the exception, so we should be particularly concerned.</div><div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><span class=""><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Also, the target attributes stuff had some correctness issues (e.g. crashes in backend) that took a while to iron out and those issues took the limelight.<br>
<br></blockquote><div><br></div></span><div>For the record, those were "crash on invalid" rather than "crash on valid".</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>-eric</div></font></span><span class=""><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D15999" rel="noreferrer" target="_blank">http://reviews.llvm.org/D15999</a><br>
<br>
<br>
<br>
</blockquote></span></div></div>
</blockquote></div><br></div></div>