<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 13, 2016 at 2:54 PM, Paul Robinson via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">probinson added a comment.<br>
<span class=""><br>
In <a href="http://reviews.llvm.org/D18706#400343" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18706#400343</a>, @probinson wrote:<br>
<br>
> Try to do fewer lookups.<br>
<br>
<br>
</span>This means every call to addLinkageName has to be guarded.  When I did the option originally (as an on/off), Eric was not a fan of guarding the calls, which is why it ended up inside the method. But now the condition is different in the two places that call the method, so maybe that's okay?<br></blockquote><div><br></div><div>A, I hadn't spotted the other call. Your call, I'm OK either way.<br><br>- Dave</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D18706" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18706</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>