<div dir="ltr">On Fri, Jan 30, 2015 at 1:06 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 20:48, Saleem Abdulrasool <<a href="mailto:compnerd@compnerd.org">compnerd@compnerd.org</a>> wrote:<br>
> I hate the clang-format output here, its not very legible IMO.  The change<br>
> here actually is to the *disabled* field.  That is, if you did<br>
> -mfpu=vfpv3-d16 and then did .fpu vfpv3 ... you need to disable the D16<br>
> behavior.  I was hoping that the commit message was sufficient.  Should I<br>
> add a comment to that effect above the table?<br>
<br>
</span>Damn, that table got me, again! The comma isn't really telling of<br>
what's on, and I assumed it was a pipe all the way through.<br>
<br>
Maybe we do need a short comment and forced formatting, to have<br>
enabled / disabled in separate lines. Something like:<br>
<br>
FPUs[] = {<br>
     {ARM::VFP,<br>
         /* enable */ ARM::FeatureVFP2,<br>
         /* disable */ ARM::FeatureNEON},<br>
     {ARM::VFPV2,<br>
         /* enable */ ARM::FeatureVFP2,<br>
         /* disable */ ARM::FeatureNEON},<br>
      ...<br></blockquote><div><br></div><div>I like this idea.  Ill go ahead and make this change soon.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Sorry for the trouble...<br>
<span class="HOEnZb"><font color="#888888"><br>
--renato<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>