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: [PATCH] PR target/65240, Fix Power{7,8} insn constraint issue with -O3 -ffast-math


On Wed, Mar 11, 2015 at 08:52:54PM -0400, David Edelsohn wrote:
> On Wed, Mar 11, 2015 at 6:21 PM, Michael Meissner
> <meissner@linux.vnet.ibm.com> wrote:
> > On Wed, Mar 11, 2015 at 01:02:06PM -0400, David Edelsohn wrote:
> >> I am concerned with the create_TOC_reference use for TARGET_TOC.  Has
> >> this been tested with big endian -mcmodel=small?
> >
> > Yes, that was a problem.  Patch coming up soon.  Thanks.
> 
> Can you call rs6000_emit_move_directly?

Well, I can, but I would have to have some sort of flag that says after the
split1 pass not to allow FP constants in move (other than 0.0).  It is doable,
but it does touch more areas in the rs6000 back end.

I am starting to think that it is just simpler to rip out all of the special
fast math handling of constants, considering the multiply by reciprocal support
has moved to SSA/tree and away from RTL.  Did you want me to investigate the
performance implications of removing it now (rather than waiting to GCC 6.0),
or just do the more limited patch that I've been pursuing.

-- 
Michael Meissner, IBM
IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA
email: meissner@linux.vnet.ibm.com, phone: +1 (978) 899-4797


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