Andreas Krebbel wrote: > this fixes a reload problem with match_dup's on commutative operands. > > Bootstrapped and regtested on x86-64, ppc64, and s390x. > > Ok? > > Bye, > > -Andreas- > > 2015-06-11 Andreas Krebbel <kreb...@linux.vnet.ibm.com> > > PR rtl-optimization/66306 > * reload.c (find_reloads): Swap the match_dup info for > commutative operands.
This does indeed appear to be broken, and the fix looks good to me. However, I'm not clear why this should be a 4.8 only patch ... the same problem seems to be still there on mainline, right? Patch is OK for mainline if it passes regression tests there. Backports to release branches are fine with me if OK with the release maintainers, after the change has been in mainline with no problems reported for a bit. Thanks, Ulrich -- Dr. Ulrich Weigand GNU/Linux compilers and toolchain ulrich.weig...@de.ibm.com
