Steve Ellcey <steve.ell...@imgtec.com> writes:
> On Mon, 2015-03-02 at 15:54 -0800, Matthew Fortune wrote:
> > Thanks for looking through and catching this.
> >
> > I had conflicting thoughts on whether the new condition should
> > reference both !ISA_HAS_FP_CONDMOVE || ISA_HAS_SEL but if we take it
> > that FP_CONDMOVE is the only way to get an integer conditional move
> > based on an FP condition then that's fine.
> >
> > > 2015-03-02  Steve Ellcey  <sell...@imgtec.com>
> > >
> > >   PR target/58158
> > >   * config/mips/mips.md (mov<mode>cc): Change ISA_HAS_SEL check to
> > >   !ISA_HAS_FP_CONDMOVE.
> >
> > OK.
> >
> > Are you planning on doing any backports of this patch to release
> > branches?
> >
> > Thanks,
> > Matthew
> 
> Given that it is very small and safe I think backporting it would make
> sense.  I will check it in on the main line today and if there are no
> problems after a few days I could check it in on the 4.8 and 4.9
> branches if that sounds good to you.

Sounds good.

Thanks,
Matthew

Reply via email to