This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH,rs6000] GCC -7, no swap optimization for vpermxor instruction
- From: Peter Bergner <bergner at vnet dot ibm dot com>
- To: Carl Love <cel at us dot ibm dot com>
- Cc: Segher Boessenkool <segher at kernel dot crashing dot org>, gcc-patches at gcc dot gnu dot org, David Edelsohn <dje dot gcc at gmail dot com>, Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>
- Date: Thu, 22 Mar 2018 13:24:08 -0500
- Subject: Re: [PATCH,rs6000] GCC -7, no swap optimization for vpermxor instruction
- References: <1521733713.4859.16.camel@us.ibm.com>
On 3/22/18 10:48 AM, Carl Love wrote:
> Backport: UNSPEC_VPERMXOR fix from mainline commit 258530:
> 2018-03-21 Carl Love <cel@us.ibm.com>
>
> * config/rs6000/r6000.c (rtx_is_swappable_p): Add case UNSPEC_VPERMXOR.
>
> gcc/testsuite/ChangeLog:
>
> 2018-03-21 Carl Love <cel@us.ibm.com>
> * gcc.target/powerpc/crypto-builtin-1-runnable.c: New test file.
Still not the right syntax. From the URL I provided you, it shows your
ChangeLog entries for your backport should look like:
gcc/ChangeLog:
2018-03-22 Carl Love <cel@us.ibm.com>
Backport from mainline
2018-03-14 Carl Love <cel@us.ibm.com>
* config/rs6000/rs6000-c.c: Add macro definitions for
ALTIVEC_BUILTIN_VEC_PERMXOR.
* config/rs6000/rs6000.h: Add #define for vec_permxor builtin.
* config/rs6000/rs6000-builtin.def: Add macro expansions for VPERMXOR.
* config/rs6000/altivec.md (altivec_vpermxor): New define expand.
* config/rs6000/rs6000-p8swap.c (rtx_is_swappable_p): Add case
UNSPEC_VPERMXOR.
* config/doc/extend.texi: Add prototypes for vec_permxor.
gcc/testsuite/ChangeLog:
2018-03-22 Carl Love <cel@us.ibm.com>
Backport from mainline
2018-03-14 Carl Love <cel@us.ibm.com>
* gcc.target/powerpc/builtins-7-runnable.c: New test file.
So basically, you cut/paste the entire ChangeLog entry from the trunk
ChangeLog file, including the date stamp (tabbed over) with the line
"Backport from mainline" added. You then add a new date stamp which
is the date you commit the patch. If you post a patch with a date stamp
and you commit it a different day, you need to update the date stamp.
Most people leave the date stamp (not the one after the Backport from
mainline) out of their patch submissions, since they don't know ahead
of time when they will commit it.
Peter