[patch, fortran][8/9/10 Regression] PR59107 Fortran : Spurious warning message with -Wsurprising

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[patch, fortran][8/9/10 Regression] PR59107 Fortran : Spurious warning message with -Wsurprising

Mark Eggleston
Please find attached three slightly different patches based on a patch
for PR59107 originally developed by Janus Weil <[hidden email]> and
Dominique d'Humieres <[hidden email]> for gcc-5. The last comment
regarding the patch was on 2015-03-21 consequently the code has moved on
somewhat and some additional changes where required resulting in 3
slightly different patches.

Tested on x86_64 using make check-fortran.

OK to commit?

Change logs for master:

fortran/ChangeLog:

     Janus Weil <[hidden email]> and
     Dominique d'Humieres  <[hidden email]>
     Mark Eggleston  <[hidden email]>

     PR59107
     * gfortran.h: Rename field resolved as resolve_symbol_called
     and assign two 2 bits instead of 1.
     * interface.c (check_dtio_interface1): Use new field name.
     (gfc_find_typebound_dtio_proc): Use new field name.
     * resolve.c (gfc_resolve_intrinsic): Replace check of the formal
     field with resolve_symbol_called is at least 2, if it is not
     set the field to 2.  (resolve_typebound_procedure): Use new field
     name.  (resolve_symbol): Use new field name and check whether it
     is at least 1, if it is not set the field to 1.

testsuite/gfortran.dg/ChangeLog:

     Mark Eggleston <[hidden email]>

     PR59107
     * gfortran.dg/pr59107.f90: New test.

The change logs for the back ports for gcc-8 and gcc-9 are included as
part of the proposed commit messages at the beginning of the patch
files. The back port dates will be updated when known.

--
https://www.codethink.co.uk/privacy.html


0001-Fortran-PR59107-master.patch (4K) Download Attachment
0001-Fortran-PR59107-gcc9.patch (4K) Download Attachment
0001-Fortran-PR59107-gcc8.patch (3K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [patch, fortran][8/9/10 Regression] PR59107 Fortran : Spurious warning message with -Wsurprising

Mark Eggleston

On 27/04/2020 09:56, Thomas Koenig wrote:

> Am 27.04.20 um 09:51 schrieb Mark Eggleston:
>> Please find attached three slightly different patches based on a
>> patch for PR59107 originally developed by Janus Weil
>> <[hidden email]> and Dominique d'Humieres <[hidden email]> for
>> gcc-5. The last comment regarding the patch was on 2015-03-21
>> consequently the code has moved on somewhat and some additional
>> changes where required resulting in 3 slightly different patches.
>>
>> Tested on x86_64 using make check-fortran.
>>
>> OK to commit?
>
> Hi Mark,
>
> OK. I just have one request: Could you specify (in a comment in the
> header) what the different values mean?
Unfortunately I don't know, they were used by Dominique d'Humieres
<[hidden email]>.
>
> Regards
>
>     Thomas
>
--
https://www.codethink.co.uk/privacy.html