Re: [PATCH,rs6000] GCC -7, no swap optimization for vpermxor instruction

2018-03-22 Thread Segher Boessenkool
Hi Carl, On Thu, Mar 22, 2018 at 08:48:33AM -0700, Carl Love wrote: > The patch adds UNSPEC_VPERMXOR to the list of instruction to not apply > swap optimization to. A new test case was added to verify the results > with -O2 match the expected results from -O0. > Backport: UNSPEC_VPERMXOR fix from

Re: [PATCH,rs6000] GCC -7, no swap optimization for vpermxor instruction

2018-03-22 Thread Segher Boessenkool
Hi Carl, On Wed, Mar 21, 2018 at 03:16:38PM -0700, Carl Love wrote: > Please let me know if the patch looks OK for the GCC 7 branch. Okay for 7 (and 6 too, if it applies there). Thanks! Segher > 2018-03-21 Carl Love > > * config/rs6000/r6000.c (rtx_is_swappable_p): Add case UNSPEC_

Re: [PATCH,rs6000] GCC -7, no swap optimization for vpermxor instruction

2018-03-22 Thread Peter Bergner
On 3/22/18 10:48 AM, Carl Love wrote: > Backport: UNSPEC_VPERMXOR fix from mainline commit 258530: > 2018-03-21  Carl Love   > > * config/rs6000/r6000.c (rtx_is_swappable_p): Add case UNSPEC_VPERMXOR. > > gcc/testsuite/ChangeLog: > > 2018-03-21  Carl Love   > * gcc.target/powerpc/cry

Re: [PATCH,rs6000] GCC -7, no swap optimization for vpermxor instruction

2018-03-22 Thread Carl Love
GCC Maintainers: This is a re-post of the patch which addresses the comments to the commit log by Peter Bergner. The following patch is a back port from mainlin of a fix for define_insn crypto_vpermxor_ in gcc/config/rs6000/crypto.md.  The issue is the vpermxor instruction does not work correctly

Re: [PATCH,rs6000] GCC -7, no swap optimization for vpermxor instruction

2018-03-21 Thread Peter Bergner
On 3/21/18 5:16 PM, Carl Love wrote: > The following patch is a back port from mainlin of a fix for [snip] > > 2018-03-21 Carl Love > > * config/rs6000/r6000.c (rtx_is_swappable_p): Add case UNSPEC_VPERMXOR. > > gcc/testsuite/ChangeLog: > > 2018-03-21 Carl Love > * gcc.target/

[PATCH,rs6000] GCC -7, no swap optimization for vpermxor instruction

2018-03-21 Thread Carl Love
GCC Maintainers: The following patch is a back port from mainlin of a fix for define_insn crypto_vpermxor_ in gcc/config/rs6000/crypto.md. The issue is the vpermxor instruction does not work correctly when the swap optimization is applied.   The issue was found as part of commit 258530 to mainli