[Bug testsuite/50023] New: FAIL: gcc.dg/graphite/id-pr46845.c (test for excess errors)

[Bug testsuite/50023] New: FAIL: gcc.dg/graphite/id-pr46845.c (test for excess errors)

[dominiq at lps dot ens.fr]

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

           Summary: FAIL: gcc.dg/graphite/id-pr46845.c (test for excess
                    errors)
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: testsuite
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: dominiq@lps.ens.fr
                CC: spop@gcc.gnu.org
              Host: powerpc-apple-darwin9
            Target: powerpc-apple-darwin9
             Build: powerpc-apple-darwin9


On powerpc-apple-darwin9 gcc.dg/graphite/id-pr46845.c fails with

Excess errors:
/opt/gcc/work/gcc/testsuite/gcc.dg/graphite/id-pr46845.c:7:1: warning: GCC
vector returned by reference: non-standard ABI extension with no compatibility
guarantee [enabled by default]
/opt/gcc/work/gcc/testsuite/gcc.dg/graphite/id-pr46845.c:6:1: warning: GCC
vector passed by reference: non-standard ABI extension with no compatibility
guarantee [enabled by default]

On i?86-*-* x86_64-*-* tha warning is disabled by

/* { dg-options "-O2 -ffast-math -fgraphite-identity -w -Wno-psabi" { target {
i?86-*-* x86_64-*-* } } } */

Why not on powerpc?

[Bug testsuite/50023] FAIL: gcc.dg/graphite/id-pr46845.c (test for excess errors)

[dominiq at lps dot ens.fr]

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

Dominique d'Humieres <dominiq at lps dot ens.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |iains at gcc dot gnu.org

--- Comment #1 from Dominique d'Humieres <dominiq at lps dot ens.fr> 2011-09-04 13:28:06 UTC ---
The test succeeds with the following patch

--- ../_gcc_clean/gcc/testsuite/gcc.dg/graphite/id-pr46845.c    2010-12-15
15:27:24.000000000 +0100
+++ gcc/testsuite/gcc.dg/graphite/id-pr46845.c    2011-09-04 15:18:55.000000000
+0200
@@ -1,4 +1,4 @@
-/* { dg-options "-O2 -ffast-math -fgraphite-identity -w -Wno-psabi" { target {
i?86-*-* x86_64-*-* } } } */
+/* { dg-options "-O2 -ffast-math -fgraphite-identity -w -Wno-psabi" { target {
i?86-*-* x86_64-*-* powerpc*-*-* } } } */

 typedef float V2SF __attribute__ ((vector_size (128)));

[Bug testsuite/50023] FAIL: gcc.dg/graphite/id-pr46845.c (test for excess errors)

[dominiq at lps dot ens.fr]

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

Dominique d'Humieres <dominiq at lps dot ens.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |grosser at gcc dot gnu.org,
                   |                            |janis at gcc dot gnu.org

--- Comment #2 from Dominique d'Humieres <dominiq at lps dot ens.fr> 2012-01-08 16:23:43 UTC ---
A patch has been submitted at
http://gcc.gnu.org/ml/gcc-patches/2011-10/msg00611.html and approved at
http://gcc.gnu.org/ml/gcc-patches/2011-10/msg00612.html . Any reason why it has
not been committed?

[Bug testsuite/50023] FAIL: gcc.dg/graphite/id-pr46845.c (test for excess errors)

[grosser at gcc dot gnu.org]

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

Tobias Grosser <grosser at gcc dot gnu.org> changed:

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

--- Comment #3 from Tobias Grosser <grosser at gcc dot gnu.org> 2012-01-09 19:20:13 UTC ---
(In reply to comment #2)
> A patch has been submitted at
> http://gcc.gnu.org/ml/gcc-patches/2011-10/msg00611.html and approved at
> http://gcc.gnu.org/ml/gcc-patches/2011-10/msg00612.html . Any reason why it has
> not been committed?

To me it seems it was committed 10.10.2011. Here the relevant commit:
http://gcc.gnu.org/viewcvs?view=revision&revision=179762

Does this commit not show up for you?

I marked this as resolved. If you cannot find the commit, please reopen the
bug.

All the best from ENS Paris
Tobi

[Bug testsuite/50023] FAIL: gcc.dg/graphite/id-pr46845.c (test for excess errors)

[dominiq at lps dot ens.fr]

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

--- Comment #4 from Dominique d'Humieres <dominiq at lps dot ens.fr> 2012-01-09 21:11:35 UTC ---
> To me it seems it was committed 10.10.2011. Here the relevant commit:
> http://gcc.gnu.org/viewcvs?view=revision&revision=179762
>
> Does this commit not show up for you?

Obviously I missed the commit: I had the patch in comment #1 in my tree since
at least 2011-09-04, so the failure no longer appeared in my radar scope (next
time I'll check more carefully).

Thanks for the pointer and sorry for the noise (at least, this PR is now
closed;-).