On 06/16/2015 07:40 PM, Ulrich Weigand wrote: > 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.
I've committed the patch after successful testing on PPC64 and s390x. I couldn't get reload working on x86_64 quickly. Bye, -Andreas-
