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

[Hal Finkel]

Lang,

Thanks!

This is not essential now, but I would like to eventually add a
test case to cover fmsubs as well.

 -Hal

On Mon, 2 Jul 2012 17:02:09 -0700
Lang Hames <lhames at gmail.com> wrote:

> 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
> >
> >



-- 
Hal Finkel
Postdoctoral Appointee
Leadership Computing Facility
Argonne National Laboratory