<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, May 18, 2014 at 11:32 AM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 18 May 2014 17:39, Saleem Abdulrasool <<a href="mailto:compnerd@compnerd.org">compnerd@compnerd.org</a>> wrote:<br>

> Author: compnerd<br>
> Date: Sun May 18 11:39:11 2014<br>
> New Revision: 209089<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=209089&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=209089&view=rev</a><br>
> Log:<br>
> ARM: make libcall setup more table driven<br>
><br>
> Rather than create a series of function calls to setup the library calls, create<br>
> a table with the information and just use the table to drive the configuration<br>
> of the library calls.  This makes it easier to both inspect the list as well as<br>
> to modify it.  NFC.<br>
<br>
</div>Hi Saleem,<br>
<br>
I agree this is an improvement, but there are other places that set<br>
libcalls names, not to mention other targets. I think such a change<br>
should have been agreed on the list by other targets' maintainers. At<br>
the very least, move the struct up the hierarchy with a<br>
table-driven-setter, like we do on the loop vectorizer's cost-table.<br></blockquote><div><br></div><div>Yes, I agree that there are other targets and places that set libcalls.  ARM and Hexagon are both particularly large in the set of library calls that they initialize.  Im not sure how many of the other targets would really benefit from such a generic infrastructure.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Can you start the conversation going on the list? No need to revert<br>
the patch, but would be good to make sure this is a design we all<br>
follow.<br></blockquote><div><br></div><div>Sure, this sounds like a good idea.  I think that getting the opinion of others on whether this is enough of a pain on other targets to warrant a generic infrastructure for this is worthwhile.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
cheers,<br>
--renato<br>
</blockquote></div><div><br></div>-- <br>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org
</div></div>