Re: [PATCH , rs6000] Add missing builtin test cases, fix arguments to match specifications.

2018-06-03 Thread Segher Boessenkool
On Sun, Jun 03, 2018 at 07:50:36PM +0200, Andreas Schwab wrote: > * gcc.target/powerpc/altivec-7.c: Remove extra brace. Please commit this if you haven't already. Thanks! Segher

Re: [PATCH , rs6000] Add missing builtin test cases, fix arguments to match specifications.

2018-06-03 Thread Andreas Schwab
* gcc.target/powerpc/altivec-7.c: Remove extra brace. Index: gcc/testsuite/gcc.target/powerpc/altivec-7.c === --- gcc/testsuite/gcc.target/powerpc/altivec-7.c(revision 261129) +++

Re: [PATCH , rs6000] Add missing builtin test cases, fix arguments to match specifications.

2018-06-01 Thread Segher Boessenkool
Hi Carl, On Tue, May 29, 2018 at 08:37:01AM -0700, Carl Love wrote: > * gcc.target/powerpc/builtins-3.c: Add tests > test_sll_vuill_vuill_vuc, Stray tab. > +/* { dg-final { scan-assembler-times "vupklpx" 1 { target le } } } */ > +/* { dg-final { scan-assembler-times "vupklpx" 1 {

Re: [PATCH , rs6000] Add missing builtin test cases, fix arguments to match specifications.

2018-05-29 Thread Carl Love
GCC maintainers: The patch has been reworked again to take advantage of the le and be selectors available, the separate BE and LE test files have been combined into a single test file. The result testing is now qualified based on the le and be selectors. This reduces the difficulty of

Re: [PATCH , rs6000] Add missing builtin test cases, fix arguments to match specifications.

2018-05-23 Thread Segher Boessenkool
On Wed, May 23, 2018 at 11:39:37AM -0700, Carl Love wrote: > > > --- a/gcc/testsuite/gcc.target/powerpc/builtins-1-be.c > > > +++ b/gcc/testsuite/gcc.target/powerpc/builtins-1-be.c > > > @@ -1,4 +1,4 @@ > > > -/* { dg-do compile { target { powerpc64-*-* } } } */ > > > +/* { dg-do compile { target

Re: [PATCH , rs6000] Add missing builtin test cases, fix arguments to match specifications.

2018-05-23 Thread Carl Love
On Wed, 2018-05-23 at 13:26 -0500, Segher Boessenkool wrote: > Hi Carl, > > I committed the be/le selectors. > > On Mon, May 21, 2018 at 08:15:30AM -0700, Carl Love wrote: > > --- a/gcc/testsuite/gcc.target/powerpc/builtins-1-be.c > > +++ b/gcc/testsuite/gcc.target/powerpc/builtins-1-be.c > > @@

Re: [PATCH , rs6000] Add missing builtin test cases, fix arguments to match specifications.

2018-05-23 Thread Segher Boessenkool
Hi Carl, I committed the be/le selectors. On Mon, May 21, 2018 at 08:15:30AM -0700, Carl Love wrote: > --- a/gcc/testsuite/gcc.target/powerpc/builtins-1-be.c > +++ b/gcc/testsuite/gcc.target/powerpc/builtins-1-be.c > @@ -1,4 +1,4 @@ > -/* { dg-do compile { target { powerpc64-*-* } } } */ > +/* {

Re: [PATCH , rs6000] Add missing builtin test cases, fix arguments to match specifications.

2018-05-21 Thread Carl Love
GCC maintainers: I updated the CommitLog for gcc/testsuite/gcc.target/powerpc/altivec- 12.c to clarify the change. A new test selector for big endian (be) and little endian (le) is added to specify the platform for the tests to run on independent of the platform being 32-bit or 64 bit. The

Re: [PATCH , rs6000] Add missing builtin test cases, fix arguments to match specifications.

2018-05-18 Thread Segher Boessenkool
On Thu, May 17, 2018 at 02:40:11PM -0700, Carl Love wrote: > On Thu, 2018-05-17 at 15:31 -0500, Segher Boessenkool wrote: > > On Wed, May 16, 2018 at 12:53:13PM -0700, Carl Love wrote: > > > diff --git a/gcc/testsuite/gcc.target/powerpc/altivec-12.c > > >

Re: [PATCH , rs6000] Add missing builtin test cases, fix arguments to match specifications.

2018-05-17 Thread Carl Love
On Thu, 2018-05-17 at 15:31 -0500, Segher Boessenkool wrote: > Hi! > > On Wed, May 16, 2018 at 12:53:13PM -0700, Carl Love wrote: > > diff --git a/gcc/testsuite/gcc.target/powerpc/altivec-12.c > > b/gcc/testsuite/gcc.target/powerpc/altivec-12.c > > index b0267b5..1f3175f 100644 > > ---

Re: [PATCH , rs6000] Add missing builtin test cases, fix arguments to match specifications.

2018-05-17 Thread Peter Bergner
On 5/17/18 3:31 PM, Segher Boessenkool wrote: > On Wed, May 16, 2018 at 12:53:13PM -0700, Carl Love wrote: >> @@ -27,21 +27,21 @@ >> /* { dg-final { scan-assembler-times "vmulosb" 1 } } */ >> >> // For LE platforms P9 and later, we generate the lxv insn instead of >> lxvd2x. >> -/* { dg-final

Re: [PATCH , rs6000] Add missing builtin test cases, fix arguments to match specifications.

2018-05-17 Thread Segher Boessenkool
Hi! On Wed, May 16, 2018 at 12:53:13PM -0700, Carl Love wrote: > diff --git a/gcc/testsuite/gcc.target/powerpc/altivec-12.c > b/gcc/testsuite/gcc.target/powerpc/altivec-12.c > index b0267b5..1f3175f 100644 > --- a/gcc/testsuite/gcc.target/powerpc/altivec-12.c > +++