[Bug testsuite/105147] New: New test case gcc.dg/pr105140.c introduced in r12-7979-geaaf77dd85c333 has excess errors

[Bug testsuite/105147] New: New test case gcc.dg/pr105140.c introduced in r12-7979-geaaf77dd85c333 has excess errors

[seurer at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105147

            Bug ID: 105147
           Summary: New test case gcc.dg/pr105140.c introduced in
                    r12-7979-geaaf77dd85c333 has excess errors
           Product: gcc
           Version: 12.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: testsuite
          Assignee: unassigned at gcc dot gnu.org
          Reporter: seurer at gcc dot gnu.org
  Target Milestone: ---

g:eaaf77dd85c333b116111bb1ae6c080154a4e411, r12-7979-geaaf77dd85c333
make  -k check-gcc RUNTESTFLAGS="dg.exp=gcc.dg/pr105140.c"
FAIL: gcc.dg/pr105140.c (test for excess errors)
# of unexpected failures        1

Excess errors:
/home/seurer/gcc/git/gcc-test/gcc/testsuite/gcc.dg/pr105140.c:16:3: error:
AltiVec argument passed to unprototyped function


commit eaaf77dd85c333b116111bb1ae6c080154a4e411 (HEAD, refs/bisect/bad)
Author: Richard Biener <rguenther@suse.de>
Date:   Mon Apr 4 10:20:05 2022 +0200

    middle-end/105140 - fix bogus recursion in fold_convertible_p

[Bug target/105147] New test case gcc.dg/pr105140.c introduced in r12-7979-geaaf77dd85c333 has excess errors

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105147

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |WAITING
          Component|testsuite                   |target
           Keywords|                            |rejects-valid
   Last reconfirmed|                            |2022-04-05

--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> ---
Why is this promoted to an error?  Note the mismatch between

void bar ();

bar (int i, int j, int k, V v)
{
}

is required to trigger the ICE and the testcase passes -w -Wno-psabi

If you think the target is right in rejecting this please add a

/* { dg-skip-if "" { <... proper target for altivec ...> } } */

[Bug target/105147] New test case gcc.dg/pr105140.c introduced in r12-7979-geaaf77dd85c333 has excess errors

[segher at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105147

Segher Boessenkool <segher at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|WAITING                     |NEW

--- Comment #2 from Segher Boessenkool <segher at gcc dot gnu.org> ---
This is an error from 2005 (g:4d3e6fae4dd3, r96124).  It is generic code that
makes this an error (instead of a warning), not the rs6000 backend, which just
provides the string (via the invalid_arg_for_unprototyped_fn hook).

s390 has this exact same error btw.

[Bug target/105147] New test case gcc.dg/pr105140.c introduced in r12-7979-geaaf77dd85c333 has excess errors

[rguenther at suse dot de]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105147

--- Comment #3 from rguenther at suse dot de <rguenther at suse dot de> ---
On Tue, 5 Apr 2022, segher at gcc dot gnu.org wrote:

> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105147
> 
> Segher Boessenkool <segher at gcc dot gnu.org> changed:
> 
>            What    |Removed                     |Added
> ----------------------------------------------------------------------------
>              Status|WAITING                     |NEW
> 
> --- Comment #2 from Segher Boessenkool <segher at gcc dot gnu.org> ---
> This is an error from 2005 (g:4d3e6fae4dd3, r96124).  It is generic code that
> makes this an error (instead of a warning), not the rs6000 backend, which just
> provides the string (via the invalid_arg_for_unprototyped_fn hook).
> 
> s390 has this exact same error btw.

I see.  So please add an appropriate dg-skip

[Bug target/105147] New test case gcc.dg/pr105140.c introduced in r12-7979-geaaf77dd85c333 has excess errors

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105147

--- Comment #4 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Segher Boessenkool <segher@gcc.gnu.org>:

https://gcc.gnu.org/g:c65d15d40738f3691ff1a39907a4b93e9fe5c5ae

commit r12-8025-gc65d15d40738f3691ff1a39907a4b93e9fe5c5ae
Author: Segher Boessenkool <segher@kernel.crashing.org>
Date:   Wed Apr 6 15:22:24 2022 +0000

    rs6000/testsuite: Skip pr105140.c

    This test fails with error "AltiVec argument passed to unprototyped
    function", but the code (in rs6000.c:invalid_arg_for_unprototyped_fn,
    from 2005) actually tests for any vector type argument.  It also does
    not fail on Darwin, not reflected here though.

    2022-04-06  Segher Boessenkool  <segher@kernel.crashing.org>

    gcc/testsuite/
            PR target/105147
            * gcc.dg/pr105140.c: Skip for powerpc*-*-*.

[Bug target/105147] New test case gcc.dg/pr105140.c introduced in r12-7979-geaaf77dd85c333 has excess errors

[segher at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105147

Segher Boessenkool <segher at gcc dot gnu.org> changed:

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

--- Comment #5 from Segher Boessenkool <segher at gcc dot gnu.org> ---
Fixed for rs6000.  As I mentioned on the ML, s390 may still have this same
issue.
If so, please reopen, update the target field, etc.

[Bug target/105147] New test case gcc.dg/pr105140.c introduced in r12-7979-geaaf77dd85c333 has excess errors

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105147

--- Comment #6 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Andreas Krebbel <krebbel@gcc.gnu.org>:

https://gcc.gnu.org/g:176df4ccb58689aae29511b99d60a448558ede94

commit r12-8039-g176df4ccb58689aae29511b99d60a448558ede94
Author: Andreas Krebbel <krebbel@linux.ibm.com>
Date:   Thu Apr 7 07:29:13 2022 +0200

    IBM zSystems/testsuite: PR105147: Skip pr105140.c

    pr105140.c fails on IBM zSystems with "vector argument passed to
    unprototyped function".  s390_invalid_arg_for_unprototyped_fn in
    s390.cc is triggered by that.

    gcc/testsuite/ChangeLog:

            PR target/105147
            * gcc.dg/pr105140.c: Skip for s390*-*-*.

[Bug target/105147] New test case gcc.dg/pr105140.c introduced in r12-7979-geaaf77dd85c333 has excess errors

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105147

--- Comment #7 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-11 branch has been updated by Richard Biener
<rguenth@gcc.gnu.org>:

https://gcc.gnu.org/g:69100fb86ff48926e5db5920a9c8cdba5ff111bd

commit r11-10031-g69100fb86ff48926e5db5920a9c8cdba5ff111bd
Author: Segher Boessenkool <segher@kernel.crashing.org>
Date:   Wed Apr 6 15:22:24 2022 +0000

    rs6000/testsuite: Skip pr105140.c

    This test fails with error "AltiVec argument passed to unprototyped
    function", but the code (in rs6000.c:invalid_arg_for_unprototyped_fn,
    from 2005) actually tests for any vector type argument.  It also does
    not fail on Darwin, not reflected here though.

    2022-04-06  Segher Boessenkool  <segher@kernel.crashing.org>

    gcc/testsuite/
            PR target/105147
            * gcc.dg/pr105140.c: Skip for powerpc*-*-*.

    (cherry picked from commit c65d15d40738f3691ff1a39907a4b93e9fe5c5ae)

[Bug target/105147] New test case gcc.dg/pr105140.c introduced in r12-7979-geaaf77dd85c333 has excess errors

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105147

--- Comment #8 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-11 branch has been updated by Richard Biener
<rguenth@gcc.gnu.org>:

https://gcc.gnu.org/g:29902934b4894280cfd5acc1765834f7392a4d48

commit r11-10032-g29902934b4894280cfd5acc1765834f7392a4d48
Author: Andreas Krebbel <krebbel@linux.ibm.com>
Date:   Thu Apr 7 07:29:13 2022 +0200

    IBM zSystems/testsuite: PR105147: Skip pr105140.c

    pr105140.c fails on IBM zSystems with "vector argument passed to
    unprototyped function".  s390_invalid_arg_for_unprototyped_fn in
    s390.cc is triggered by that.

    gcc/testsuite/ChangeLog:

            PR target/105147
            * gcc.dg/pr105140.c: Skip for s390*-*-*.

    (cherry picked from commit 176df4ccb58689aae29511b99d60a448558ede94)

[Bug target/105147] New test case gcc.dg/pr105140.c introduced in r12-7979-geaaf77dd85c333 has excess errors

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105147

--- Comment #9 from Richard Biener <rguenth at gcc dot gnu.org> ---
*** Bug 105724 has been marked as a duplicate of this bug. ***

[Bug target/105147] New test case gcc.dg/pr105140.c introduced in r12-7979-geaaf77dd85c333 has excess errors

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105147

--- Comment #10 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-10 branch has been updated by Richard Biener
<rguenth@gcc.gnu.org>:

https://gcc.gnu.org/g:b3157b03a8b83445e6ce48c9d9c6a6b614937691

commit r10-10769-gb3157b03a8b83445e6ce48c9d9c6a6b614937691
Author: Segher Boessenkool <segher@kernel.crashing.org>
Date:   Wed Apr 6 15:22:24 2022 +0000

    rs6000/testsuite: Skip pr105140.c

    This test fails with error "AltiVec argument passed to unprototyped
    function", but the code (in rs6000.c:invalid_arg_for_unprototyped_fn,
    from 2005) actually tests for any vector type argument.  It also does
    not fail on Darwin, not reflected here though.

    2022-04-06  Segher Boessenkool  <segher@kernel.crashing.org>

    gcc/testsuite/
            PR target/105147
            * gcc.dg/pr105140.c: Skip for powerpc*-*-*.

    (cherry picked from commit c65d15d40738f3691ff1a39907a4b93e9fe5c5ae)

[Bug target/105147] New test case gcc.dg/pr105140.c introduced in r12-7979-geaaf77dd85c333 has excess errors

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105147

--- Comment #11 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-10 branch has been updated by Richard Biener
<rguenth@gcc.gnu.org>:

https://gcc.gnu.org/g:1cccf2e43b43d6e75df2257ddc2682dceb9f04ae

commit r10-10770-g1cccf2e43b43d6e75df2257ddc2682dceb9f04ae
Author: Andreas Krebbel <krebbel@linux.ibm.com>
Date:   Thu Apr 7 07:29:13 2022 +0200

    IBM zSystems/testsuite: PR105147: Skip pr105140.c

    pr105140.c fails on IBM zSystems with "vector argument passed to
    unprototyped function".  s390_invalid_arg_for_unprototyped_fn in
    s390.cc is triggered by that.

    gcc/testsuite/ChangeLog:

            PR target/105147
            * gcc.dg/pr105140.c: Skip for s390*-*-*.

    (cherry picked from commit 176df4ccb58689aae29511b99d60a448558ede94)