<div dir="ltr">I posted proposals for the function-attribute quick fix here:<br><a href="https://reviews.llvm.org/D24815">https://reviews.llvm.org/D24815</a><br><a href="https://reviews.llvm.org/D24816">https://reviews.llvm.org/D24816</a> <br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 20, 2016 at 4:23 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 Tue, Sep 20, 2016 at 1:18 PM Sanjay Patel <<a href="mailto:spatel@rotateright.com" target="_blank">spatel@rotateright.com</a>> wrote:<br></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_extra"><div class="gmail_quote">On Tue, Sep 20, 2016 at 1:43 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><div class="gmail_quote"><div>b) The non-x86 implementations of this break LTO.</div></div></div></blockquote><div><br></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Oh wow...sorry for missing that. Is there a link to that bug/discussion? I'm curious why x86 is immune.<br></div></div></div></div></blockquote><div><br></div></span><div>I explained it upthread, but x86 is basically immune here because you weren't basing the values on any particular subtarget.</div><span class=""><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_extra"><div class="gmail_quote"><div></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br> </div><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></span><div>Can you at least just start attaching it to the function as an option for now?</div><span></span></div></div>
</blockquote></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"></div><br></div><div class="gmail_extra">Sure - I'll take a look at that as soon as I can.<br></div><div class="gmail_extra"><br></div></div></blockquote><div><br></div></span><div>Thanks!</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>-eric </div></font></span></div></div>
</blockquote></div><br></div>