Hi Rich,<br><br>This all looks fine to me. <br><br>Cheers,<br><br>James<br><div class="gmail_quote">On Fri, 28 Nov 2014 at 17:08, Richard Barton <<a href="mailto:richard.barton@arm.com">richard.barton@arm.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi reviewers<br>
<br>
Attached patches add missing arguments to the -mfpu option for ARM.<br>
<br>
The current set of supported options misses out neon-vfpv4. The first patch<br>
addresses this omission.<br>
<br>
I would also like to add a corresponding neon-vfpv3 argument. Note that this<br>
is not supported by GCC, but follows the same pattern for GCC options. This<br>
is patch number 2.<br>
<br>
I have split the patch in two as hopefully patch 1 will cause no<br>
controversy.<br>
<br>
Please review.<br>
<br>
Thanks<br>
Rich<br>
______________________________<u></u>_________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/cfe-commits</a><br>
</blockquote></div>