* Thomas Schwinge:

> 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?

Sorry about that.  My change was invalid, I swapped int/void by
accident.  Too many edits in too short a period.  I will fix it as
obvious.

Thanks,
Florian

Reply via email to