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