On Thu, Jul 15, 2021 at 11:25 AM Bill Schmidt <wschm...@linux.ibm.com> wrote: > > Hi, > > Segher preapproved this patch in https://gcc.gnu.org/PR101129. It differs > slightly from what was posted there, needing an additional test to ensure the > insn is a SET. The patch also includes the test case provided by the OP. > Bootstrap and regtest succeeded on P9 little-endian. > > This bug has been around a long time, so the fix should be backported to all > open releases. Is this okay after some burn-in time? > > Thanks! > Bill > > rs6000: Don't let swaps pass break multiply low-part (PR101129) > > 2021-07-15 Bill Schmidt <wschm...@linux.ibm.com> > > gcc/ > * config/rs6000/rs6000-p8swap.c (has_part_mult): New. > (rs6000_analyze_swaps): Insns containing a subreg of a mult are > not swappable. > > gcc/testsuite/ > * gcc.target/powerpc/pr101129.c: New.
Thanks for fixing this so quickly. Okay everywhere. Thanks, David
