<div dir="ltr">On Fri, Jan 30, 2015 at 12:36 PM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 30 January 2015 at 19:35, Saleem Abdulrasool <<a href="mailto:compnerd@compnerd.org">compnerd@compnerd.org</a>> wrote:<br>
> +    {ARM::VFPV3, ARM::FeatureVFP2 | ARM::FeatureVFP3,<br>
> +     ARM::FeatureNEON | ARM::FeatureD16},<br>
<br>
</span>Hi Saleem,<br>
<br>
That's wrong. vfpv3 == vfpv3-d32, that's why you have to specify<br>
vfpv3-d16 for the shortened variant.<br></blockquote><div><br></div><div>Correct :-).</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The same goes for vfpv4, neon, etc. Your original patch was correct.<br></blockquote><div><br></div><div>I hate the clang-format output here, its not very legible IMO.  The change here actually is to the *disabled* field.  That is, if you did -mfpu=vfpv3-d16 and then did .fpu vfpv3 ... you need to disable the D16 behavior.  I was hoping that the commit message was sufficient.  Should I add a comment to that effect above the table?</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><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org</div>
</div></div>