<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Apr 2, 2018, 5:57 PM Rafael Avila de Espindola via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">espindola requested changes to this revision.<br>
espindola added a comment.<br>
This revision now requires changes to proceed.<br>
<br>
>> Given that we have nonlazybind on each GV, if we are replacing the printf GV with the puts GV we should copy nonlazybind from one to the other.<br>
><br>
> This is not happening for any attribute of printf right now and some attributes simply cannot be copied over so this doesn't seem easy to do.<br>
<br>
That just means you found the first case where we have to do it. Being hard is also not a reason for doing the wrong thing.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">That's not what I implied! It is not clear to me how you would go about doing this and if this is worth it at all. There are attributes you should not copy over and reason about what are the common attributes. If you have a plan here I am all ears , :) thanks!</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="https://reviews.llvm.org/D45180" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D45180</a><br>
<br>
<br>
<br>
</blockquote></div></div></div>