Hi! On Thu, Jan 09, 2020 at 06:52:05PM -0500, Michael Meissner wrote: > 2020-01-09 Michael Meissner <meiss...@linux.ibm.com> > > * config/rs6000/rs6000.c (rs6000_adjust_vec_address): Add some > gcc_asserts.
> + /* All insns should use the 'Q' constraint (address is a single > register) > + if the element number is not a constant. */ > + gcc_assert (REG_P (addr) || SUBREG_P (addr)); So maybe you should just more directly say gcc_assert (satisfies_constraint_Q (addr)); ? The Q constraint does not allow subregs, btw, is that an oversight? Okay for trunk either way. Thanks! Segher