On 07/13/2017 02:59 AM, Richard Sandiford wrote: > This patch uses scalar_int_modes for: > > /* If I2 is setting a pseudo to a constant and I3 is setting some > sub-part of it to another constant, merge them by making a new > constant. */ > > This was already implicit, but the danger with checking only > CONST_SCALAR_INT_P is that it can include CC values too. > > 2017-07-13 Richard Sandiford <[email protected]> > Alan Hayward <[email protected]> > David Sherwood <[email protected]> > > gcc/ > * combine.c (try_combine): Use is_a <scalar_int_mode> when > trying to combine a full-register integer set with a subreg > integer set. OK. I briefly mis-read the SET_DEST in the is_a as a SET_SRC (which in this context could have VOIDmode). But you're getting the mode from the dest :-)
jeff
