On 11/15/17 9:40 AM, Aaron Sawdey wrote: > Index: gcc/rtlanal.c > =================================================================== > --- gcc/rtlanal.c (revision 254553) > +++ gcc/rtlanal.c (working copy) > @@ -5623,7 +5623,11 @@ > if (CC0_P (op0)) > return 0; > > - return gen_rtx_fmt_ee (code, VOIDmode, op0, op1); > + /* We promised to return a comparison. */ > + rtx ret = gen_rtx_fmt_ee (code, VOIDmode, op0, op1); > + if (COMPARISON_P (ret)) > + return ret; > + return 0;
I have no input on whether this approach is correct or not, but... I know the return above this returns 0 as do other locations in the file, but new code should return NULL_RTX. Peter