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