Hi Florian!

Thanks for all your ongoing clean-up work!

On 2023-11-10T23:07:55+0100, Florian Weimer <fwei...@redhat.com> wrote:
> This change updates the gcc.c-torture/execute/ to avoid obsolete
> language constructs.  In the changed tests, use of the features
> appears to be accidental, and updating allows the tests run with
> the default compiler flags.

> --- a/gcc/testsuite/gcc.c-torture/execute/931004-13.c
> +++ b/gcc/testsuite/gcc.c-torture/execute/931004-13.c

> @@ -42,7 +43,8 @@ f (int n, struct tiny x, struct tiny y, struct tiny z, long 
> l)
>      abort ();
>  }
>
> -main ()
> +void
> +main (int)
>  {
>    struct tiny x[3];
>    x[0].c = 10;

The nvptx back end doesn't like that one:

    PASS: gcc.c-torture/execute/931004-13.c   -O0  (test for excess errors)
    [-PASS:-]{+FAIL:+} gcc.c-torture/execute/931004-13.c   -O0  execution test
    [...]

    error   : Prototype doesn't match for 'main' [...]

I'll add handling for this case in (I suppose)
'gcc/config/nvptx/nvptx.cc:write_fn_proto_1' if that indeed is a
conforming declaration of 'main', but is it really?


Grüße
 Thomas
-----------------
Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 
München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas 
Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht 
München, HRB 106955

Reply via email to