[Patch, MIPS] Fix PR 58158 - ICE on MIPS Loongson

Steve Ellcey sellcey@imgtec.com
Tue Mar 3 00:02:00 GMT 2015


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  <sellcey@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
sellcey@imgtec.com



More information about the Gcc-patches mailing list