<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Sep 7, 2016 at 11:14 AM, Anton Korobeynikov <span dir="ltr"><<a href="mailto:anton@korobeynikov.info" target="_blank">anton@korobeynikov.info</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><span class="gmail-">> -/// InitLibcallCallingConvs - Set default libcall CallingConvs.<br>
> -///<br>
> -static void InitLibcallCallingConvs(<wbr>CallingConv::ID *CCs) {<br>
> -  for (int i = 0; i < RTLIB::UNKNOWN_LIBCALL; ++i) {<br>
> -    CCs[i] = CallingConv::C;<br>
> -  }<br>
> +/// Set default libcall CallingConvs.<br>
> +static void InitLibcallCallingConvs(<wbr>CallingConv::ID *CCs, const Triple &T) {<br>
> +  for (int LC = 0; LC < RTLIB::UNKNOWN_LIBCALL; ++LC)<br>
> +    CCs[LC] = CallingConv::C;<br>
> +<br>
> +  // The builtins on ARM always use AAPCS, irrespective of wheter C is AAPCS or<br>
> +  // AAPCS_VFP.<br>
> +  if (T.getArch() == Triple::arm || T.getArch() == Triple::thumb)<br>
</span>That's gross hack in the generic target lowering code... Why can't you<br>
initialize the CC from the backend?</blockquote><div><br></div><div>Yeah, I agree.  This was just a complete oversight on my part.  Addressed in SVN r281088.</div><div><br></div><div>Thanks for pointing this out!</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><span class="gmail-HOEnZb"><font color="#888888"><br>
--<br>
With best regards, Anton Korobeynikov<br>
Department of Statistical Modelling, Saint Petersburg State University<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org</div>
</div></div>