Do you not see an attached patch? That's bizarre - gmail shows it attached in my sent messages.<div><br></div><div>Attaching again just in case.</div><div><br></div><div>- Lang.<br><br><div class="gmail_quote">On Mon, Jul 2, 2012 at 4:52 PM, Meador Inge <span dir="ltr"><<a href="mailto:meadori@codesourcery.com" target="_blank">meadori@codesourcery.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Attaching the patch might help the review ;-)<br>
<div><div class="h5"><br>
On Jul 2, 2012, at 4:43 PM, Lang Hames wrote:<br>
<br>
> Hi All,<br>
><br>
> The attached patch adds a -ffp-contract=<style> option which (mostly) follows the behavior of GCC's option. Styles are 'fast', for aggressive FMA formation, 'on' for FP_CONTRACT compliance, and 'off' for no FMA formation (in the future this might be relaxed to FMA formation in cases where the result is provably unaffected).<br>

><br>
> Feedback would be welcome. In particular, I have toggled the fp-contract style based on the ffast-math option in Clang::ConstructJob(...) in lib/Driver/Tools.cpp. Is that the right place to do that kind of option fiddling? (It seems to be where the other fast-math related floating point options get handled).<br>

><br>
> Cheers,<br>
> Lang.<br>
<br>
<br>
</div></div>--<br>
Meador Inge<br>
CodeSourcery / Mentor Embedded<br>
<a href="http://www.mentor.com/embedded-software" target="_blank">http://www.mentor.com/embedded-software</a><br>
<br>
</blockquote></div><br></div>