[Bug target/60076] New: gcc.dg/vect/pr60012.c FAILs on Solaris/SPARC

[Bug target/60076] New: gcc.dg/vect/pr60012.c FAILs on Solaris/SPARC

[ro at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60076

            Bug ID: 60076
           Summary: gcc.dg/vect/pr60012.c FAILs on Solaris/SPARC
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: ro at gcc dot gnu.org
                CC: ebotcazou at gcc dot gnu.org, rguenth at gcc dot gnu.org
              Host: sparc*-sun-solaris2.*
            Target: sparc*-sun-solaris2.*
             Build: sparc*-sun-solaris2.*

The new gcc.dg/vect/pr60012.c FAILs on Solaris/SPARC, both 32 and 64-bit:

FAIL: gcc.dg/vect/pr60012.c scan-tree-dump vect "LOOP VECTORIZED"
FAIL: gcc.dg/vect/pr60012.c -flto -ffat-lto-objects  scan-tree-dump vect "LOOP
VECTORIZED"

I'm attaching the .vect dump.

  Rainer

[Bug target/60076] gcc.dg/vect/pr60012.c FAILs on Solaris/SPARC

[ro at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60076

Rainer Orth <ro at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.9.0

[Bug target/60076] gcc.dg/vect/pr60012.c FAILs on Solaris/SPARC

[ro at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60076

--- Comment #1 from Rainer Orth <ro at gcc dot gnu.org> ---
Created attachment 32047
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=32047&action=edit
.vect dump

[Bug target/60076] gcc.dg/vect/pr60012.c FAILs on Solaris/SPARC

[rguenth at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60076

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |WAITING
   Last reconfirmed|                            |2014-02-05
           Assignee|unassigned at gcc dot gnu.org      |rguenth at gcc dot gnu.org
     Ever confirmed|0                           |1

--- Comment #2 from Richard Biener <rguenth at gcc dot gnu.org> ---
Mine.  Some dg-requires of vect_XXX missing probably.

note: not vectorized: no vectype for stmt: _8 = *_7;
 scalar_type: long long unsigned int

I propose to change s/unsigned long long/unsigned int/g in the testcase.  Does
that fix it for you?

[Bug target/60076] gcc.dg/vect/pr60012.c FAILs on Solaris/SPARC

[ro at CeBiTec dot Uni-Bielefeld.DE]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60076

--- Comment #3 from ro at CeBiTec dot Uni-Bielefeld.DE <ro at CeBiTec dot Uni-Bielefeld.DE> ---
> --- Comment #2 from Richard Biener <rguenth at gcc dot gnu.org> ---
> Mine.  Some dg-requires of vect_XXX missing probably.
>
> note: not vectorized: no vectype for stmt: _8 = *_7;
>  scalar_type: long long unsigned int
>
> I propose to change s/unsigned long long/unsigned int/g in the testcase.  Does
> that fix it for you?

Unfortunately not.  I'm attaching the new .vect file.

    Rainer

[Bug target/60076] gcc.dg/vect/pr60012.c FAILs on Solaris/SPARC

[ro at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60076

--- Comment #4 from Rainer Orth <ro at gcc dot gnu.org> ---
Created attachment 32052
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=32052&action=edit
unsigned int .vect dump

[Bug target/60076] gcc.dg/vect/pr60012.c FAILs on Solaris/SPARC

[rguenth at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60076

--- Comment #5 from Richard Biener <rguenth at gcc dot gnu.org> ---
note: extract even/odd not supported by target

aha...  next try:

Index: gcc/testsuite/gcc.dg/vect/pr60012.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/pr60012.c (revision 207504)
+++ gcc/testsuite/gcc.dg/vect/pr60012.c (working copy)
@@ -8,14 +8,14 @@ typedef struct
 } complex16_t;

 void
-libvector_AccSquareNorm_ref (unsigned long long  *acc,
+libvector_AccSquareNorm_ref (unsigned int *acc,
                             const complex16_t *x, unsigned len)
 {
   unsigned i;
   for (i = 0; i < len; i++)
-    acc[i] += ((unsigned long long)((int)x[i].real * x[i].real))
-       + ((unsigned long long)((int)x[i].imag * x[i].imag));
+    acc[i] += ((unsigned int)((int)x[i].real * x[i].real))
+       + ((unsigned int)((int)x[i].imag * x[i].imag));
 }

-/* { dg-final { scan-tree-dump "LOOP VECTORIZED" "vect" } } */
+/* { dg-final { scan-tree-dump "LOOP VECTORIZED" "vect" { target {
vect_extract_even_odd } } } } */
 /* { dg-final { cleanup-tree-dump "vect" } } */

[Bug target/60076] gcc.dg/vect/pr60012.c FAILs on Solaris/SPARC

[ro at CeBiTec dot Uni-Bielefeld.DE]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60076

--- Comment #6 from ro at CeBiTec dot Uni-Bielefeld.DE <ro at CeBiTec dot Uni-Bielefeld.DE> ---
> --- Comment #5 from Richard Biener <rguenth at gcc dot gnu.org> ---
> note: extract even/odd not supported by target
>
> aha...  next try:

This works: scan-tree-dump isn't even attempted.

Thanks.
        Rainer

[Bug target/60076] gcc.dg/vect/pr60012.c FAILs on Solaris/SPARC

[rguenth at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60076

--- Comment #8 from Richard Biener <rguenth at gcc dot gnu.org> ---
Author: rguenth
Date: Wed Feb  5 15:14:56 2014
New Revision: 207509

URL: http://gcc.gnu.org/viewcvs?rev=207509&root=gcc&view=rev
Log:
2014-02-05  Richard Biener  <rguenther@suse.de>

    PR testsuite/60076
    * gcc.dg/vect/pr60012.c: Require vect_extract_even_odd and
    avoid using unsigned long long.

Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gcc.dg/vect/pr60012.c

[Bug target/60076] gcc.dg/vect/pr60012.c FAILs on Solaris/SPARC

[rguenth at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60076

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|WAITING                     |RESOLVED
         Resolution|---                         |FIXED

--- Comment #7 from Richard Biener <rguenth at gcc dot gnu.org> ---
Fixed.