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 <richard.sandif...@linaro.org> > Alan Hayward <alan.hayw...@arm.com> > David Sherwood <david.sherw...@arm.com> > > 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
