Re: [Patch, fortran] PR98897 - Erroneous procedure attribute for associate name

2021-02-12 Thread Paul Richard Thomas via Gcc-patches
Hi All, Following off-list discussion with Tobias, I have committed the patch as submitted to 10- and 11-branches. A rather general problem with parsing and matching, which arose from the discussion, has been shunted into PR99065. If possible, I intend to fix this by two pass parsing/matching of

Re: [Patch, fortran] PR98897 - Erroneous procedure attribute for associate name

2021-02-02 Thread Tobias Burnus
Hi, first, I have attached a new example – it works if I move bar/hello up, but if 'foo' comes first, it fails. I think it is valid. (ifort 19 also compiles it.) Sorry for trying hard to find examples where it does not work – but I have simply the feeling that resolving things during parsing can

Re: [Patch, fortran] PR98897 - Erroneous procedure attribute for associate name

2021-02-02 Thread Paul Richard Thomas via Gcc-patches
Hi Tobias, ifort (IFORT) 2021.1 Beta 20201112 is happy with the testcase in the patch. In foo.f90, if I remove call var(i) ! { dg-bogus "VARIABLE attribute of 'var' conflicts with PROCEDURE attribute" } gfortran correctly complains 23 | associate (var => bar()) |

Re: [Patch, fortran] PR98897 - Erroneous procedure attribute for associate name

2021-02-02 Thread Tobias Burnus
Hi Paul, On 02.02.21 13:20, Paul Richard Thomas via Gcc-patches wrote: This is more or less 'obvious' and does not require any further explanation. Well, I am not sure whether calling resolve is premature or not. In any case, it still fails for the attached testcase. (Related but separate issu

[Patch, fortran] PR98897 - Erroneous procedure attribute for associate name

2021-02-02 Thread Paul Richard Thomas via Gcc-patches
This is more or less 'obvious' and does not require any further explanation. Regtests with FC33/x86_64 - OK for master (and )? Paul Fortran: Fix calls to associate name typebound subroutines [PR98897]. 2021-02-02 Paul Thomas gcc/fortran PR fortran/98897 * match.c (gfc_match_call): Inclu