<div dir="ltr">I was thinking about implementing the mechanism to add function attributes in llvm/CodeGen/CommandFlags.h so that both llc and opt can benefit from it and we don't have to rely on funcattrs to add attributes. The downside is that we won't be able to remove attributes from clang's command line.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 18, 2015 at 8:42 AM, James Molloy <span dir="ltr"><<a href="mailto:james@jamesmolloy.co.uk" target="_blank">james@jamesmolloy.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Akira,<br><br>This will be useful whenever functionattrs is run; this includes clang but not, I think, llc. <br><br>I considered adding an option to remove attributes, but I decided against it. Adding an attribute is easy - attributes are almost never removed during compilation (apart from inlining).  However attributes are regularly inferred, so I could really make no guarantee about the effectiveness of removing a function attribute (as it may be re added later!)<span class="HOEnZb"><font color="#888888"><br><br>James<br></font></span><div class="gmail_quote"><div><div class="h5"><div dir="ltr">On Wed, 18 Nov 2015 at 15:55, Akira Hatanaka via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br></div></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">ahatanak added a subscriber: ahatanak.<br>
ahatanak added a comment.<br>
<br>
Will this command line option be useful for llc or other tools too? Also, an option to remove the attributes from a function might be useful too.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D14775" rel="noreferrer" target="_blank">http://reviews.llvm.org/D14775</a><br>
<br>
<br>
<br></div></div><span class="">
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">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>
</span></blockquote></div>
</blockquote></div><br></div>