This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Ping Re: Add option for whether ceil etc. can raise "inexact", adjust x86 conditions


On Thu, 2 Jun 2016, Bernd Schmidt wrote:

> On 06/02/2016 02:00 PM, Jan Hubicka wrote:
> > > Ping.  This patch
> > > <https://gcc.gnu.org/ml/gcc-patches/2016-05/msg02131.html> is pending
> > > review (for the non-x86-specific parts).
> > The inliner bits looks fine to me.
> 
> In case that leaves anything unapproved, the remaining parts are OK too,
> modulo one question - shouldn't this option be added to the set enabled by
> -funsafe-math-optimizations? It looks like one pattern in i386.md used to be
> enabled by this option and now is no longer.

-funsafe-math-optimizations implies -fno-trapping-math which causes this 
option to have no effect (the difference between -ffp-int-builtin-inexact 
and -fno-fp-int-builtin-inexact is only meaningful if -ftrapping-math, 
since it relates to the raising of exceptions).  The patterns testing this 
option all test (flag_fp_int_builtin_inexact || !flag_trapping_math).

-- 
Joseph S. Myers
joseph@codesourcery.com


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]