Re: [PATCH, rs6000] Add missing builtin test cases

2018-06-28 Thread Segher Boessenkool
Hi! On Wed, Jun 27, 2018 at 03:14:18PM -0700, Carl Love wrote: > The following patch adds missing test cases for > vec_extract_fp32_from_shortl(), vec_extract_fp32_from_shorth(), and > vec_extract(). This is fine for trunk. Thank you! Segher > 2018-06-27 Carl Love > > * gcc.targe

[PATCH, rs6000] Add missing builtin test cases

2018-06-27 Thread Carl Love
GCC Maintainers: The following patch adds missing test cases for vec_extract_fp32_from_shortl(), vec_extract_fp32_from_shorth(), and vec_extract(). The patch has been run on:   powerpc64le-unknown-linux-gnu (Power 8 LE)   powerpc64le-unknown-linux-gnu (Power 9 LE) without regressions.   Pl

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) +++ gcc/testsuite/gcc.target/powerpc/altiv

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 { tar

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 maintainin

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 vario

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 > > > b/gcc/testsuite/gcc.target/powerpc/altivec-

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 > > --- a/gcc/tests

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 > +++ b/gcc/testsuite/gcc.target/powe

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

2018-05-16 Thread Carl Love
GCC maintainers: The following patch adds various missing builtin test cases. I also went through the various test files and made sure that each test had a corresponding instruction count test if appropriate. In some cases, I had to add count tests. For one of the tests, I had to create a BE an