<div style="font-family: arial, helvetica, sans-serif"><font size="2"><div class="gmail_quote">On Wed, Jun 20, 2012 at 11:43 PM, Lang Hames <span dir="ltr"><<a href="mailto:lhames@gmail.com" target="_blank">lhames@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi All,<div><br></div><div>This patch adds support for gcc's -fexcess-precision flag to clang. I'm new to clang's option handling. Could somebody please sanity check it for me?</div>
</blockquote><div><br></div><div>A high level point:</div><div><br></div><div>The flag semantics '-fexcess-precision' (as spec'ed for GCC) don't seem to map clearly onto this use case, especially as John argued so effectively in the previous email thread. I feel like this flag should remain very specifically attached to FP operations with more bits, but none of the asymmetrical properties of FMAs, so that there is no confusion between Clang's semantics and GCC's.</div>
<div><br></div><div>I'd suggest a new flag without any baggage, and one that perhaps ties more closely into the idea of contracted expressions, or some other concept intrinsic to fusing operations. Maybe '-fcontract-expressions={fast,standard,strict}'? Dunno. Naming things is hard.</div>
<div><br></div><div><br></div><div>Anyways, details about the patch:</div><div><br></div><div>- You should integrate the forwarding of this with the -ffast-math section more fully -- -ffast-math and maybe some other flags should synthesize the 'fast' variation.</div>
<div><br></div><div>- You need to handle serializing this flag in the frontend.</div><div><br></div><div>- Maybe add a help section to the flag?</div></div></font></div>