<br><br><div class="gmail_quote">On Thu, Jan 19, 2012 at 10:16 AM, Kostya Serebryany <span dir="ltr"><<a href="mailto:kcc@google.com">kcc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br><br><div class="gmail_quote"><div class="im">On Thu, Jan 19, 2012 at 10:14 AM, Kaelyn Uhrain <span dir="ltr"><<a href="mailto:rikka@google.com" target="_blank">rikka@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="gmail_quote"><div>On Thu, Jan 19, 2012 at 9:56 AM, Kostya Serebryany <span dir="ltr"><<a href="mailto:kcc@google.com" target="_blank">kcc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<br><br><div class="gmail_quote">On Thu, Jan 19, 2012 at 1:18 AM, Anton Korobeynikov <span dir="ltr"><<a href="mailto:anton@korobeynikov.info" target="_blank">anton@korobeynikov.info</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



Hi Kostya,<br>
<div><br>
<br>
> The patches are attached, the llvm patch is also published<br>
> here: <a href="http://codereview.appspot.com/5544058/" target="_blank">http://codereview.appspot.com/5544058/</a><br>
</div>What is the performance impact of this?<br></blockquote><div><br></div><div>Good point (I assume you mean compile-time performance, not the performance of compiled programs).</div><div>I've verified that the compile-time did not change by building 403.gcc and 483.xalancbmk with the original and modified clang with -O1. </div>


</div></blockquote><div><br></div></div><div>Out of curiosity did you check the compile-time performance for 64-bit clang/llvm, </div></div></blockquote><div><br></div></div><div>64-bit. </div><div>On 32-bit there may be some change indeed, but we still need more attributes, right? :) </div>
</div></blockquote><div><br></div><div>Their necessity would be all the more reason to understand the performance impact on a 32-bit compiler. :)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="gmail_quote"><div class="im">
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><div>32-bit clang/llvm, or both?  I would imagine that extending attributes to 64-bit would have negligible impact for a 64-bit compiler binary but potentially greater impact for a 32-bit one as they would no longer fit in a single register.</div>


<div></div></div><br><div>Cheers,</div><div>Kaelyn</div>
</blockquote></div></div><br>
</blockquote></div><br>