<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 1, 2015 at 11:39 AM, Ahmed Bougacha <span dir="ltr"><<a href="mailto:ahmed.bougacha@gmail.com" target="_blank">ahmed.bougacha@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Apr 1, 2015 at 10:07 AM, Pirama Arumuga Nainar<br>
<<a href="mailto:pirama@google.com">pirama@google.com</a>> wrote:<br>
> I do not have a reasonable way of finding whether ISD::FP_TO_FP16, ISD::FP16_TO_FP are supported by the target. If unsupported, we have to fall back to making a libcall to gnu_h2f_ieee. Right now, this is a hard-coded branch in LegalizeFloatTypes.cpp:GetPromotedValue but I need suggestions on how to handle this.<br>
<br>
</span>Also, now that I think about it, isn't this already handled by ops<br>
legalization?  I believe you shouldn't have to worry about unsupported<br>
operations when legalizing types.<br></blockquote><div><br></div><div>It is not handled for Mips.  So, ISEL aborts if FP_TO_FP16 is used instead of the libcall.  I wasn't aware that Op legalization handles this.  I'll find out why Mips doesn't do the right thing here.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
-Ahmed<br>
</font></span></blockquote></div><br></div></div>