<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 8, 2013 at 12:33 PM, Hal Finkel <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank" class="cremed">hfinkel@anl.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb adM"><div class="im">> Okay, thanks! (Maybe this leaves me with InstCombine?) Does anything<br>
> else use, or is expected to use, the cold attribute directly in<br>
> addition to BPI? If not, maybe leaving this in BPI is just as good<br>
> as anything else.<br>
<br>
</div></div>Chandler, please express an opinion. Would you want this in InstCombine? Thanks!</blockquote></div><br>Sorry, missed this email.</div><div class="gmail_extra"><br></div><div class="gmail_extra">I agree with your analysis that FunctionAttrs is the wrong place. Sorry for that.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">I think I would put this in SimplifyLibCalls much as Meador Inge has said now. That's where we make assumptions about library calls having special meaning, and this makes a lot of sense to me as being part of that. And this would in turn run inside of InstCombine.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">-Chandler</div></div>