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.

Steve Ellcey
sell...@imgtec.com

Reply via email to