[cfe-commits] [PATCH] Add -ffp-contract option.

Lang Hames lhames at gmail.com
Mon Jul 2 17:02:09 PDT 2012


Do you not see an attached patch? That's bizarre - gmail shows it attached
in my sent messages.

Attaching again just in case.

- Lang.

On Mon, Jul 2, 2012 at 4:52 PM, Meador Inge <meadori at codesourcery.com>wrote:

> Attaching the patch might help the review ;-)
>
> On Jul 2, 2012, at 4:43 PM, Lang Hames wrote:
>
> > Hi All,
> >
> > 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).
> >
> > 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).
> >
> > Cheers,
> > Lang.
>
>
> --
> Meador Inge
> CodeSourcery / Mentor Embedded
> http://www.mentor.com/embedded-software
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120702/647c6120/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: clang_ffp-contract.patch
Type: application/octet-stream
Size: 6762 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120702/647c6120/attachment.obj>


More information about the cfe-commits mailing list