<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Oct 31, 2013 at 7:29 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="im">----- Original Message -----<br>
> ----- Original Message -----<br>
> ><br>
> > Actually looking at this patch, why not do this in the pass which<br>
> > adds function attributes by adding the cold attribute to error<br>
> > reporting library functions?<br>
<br>
</div>Unless we're willing to mark all calls to fprintf as cold (which I don't think that we want to do), then I can't add the attributes to the function declarations. Only noreturn, nounwind, readonly and readnone are currently supported as call-site function attributes. Being able to mark particular call sites as cold does seem to make sense, so maybe this is worth extending?<br>
</blockquote><div><br></div><div>That had been my thought.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
You're specifically referring to IPO/FunctionAttrs.cpp?<br></blockquote><div><br></div><div>Yep.</div></div></div></div>