On 10.04.21 22:45, José Rui Faustino de Sousa via Fortran wrote:
On 10/04/21 17:37, Tobias Burnus wrote:
And you need an additional single-line summary for git – which should
be part of the patch submission.
In case you are waiting for me, I did write:
'LGTM – Thanks for the patch. Two nits:'
and expected that you just fixed them when committing the patch.
(Actually, as they were tiny comments and to the commit log only,
I did not even expect a posting of the committed patch or its commit log.)
But as you didn't commit it it yet, I gathered you are waiting for me.
Hence, you get another LGTM from me :-)
Thanks again for the patch – and the other patches (which still need to
be reviewed).
Tobias
Fortran: Fix ICE due to referencing a NULL pointer [PR100018]
gcc/fortran/ChangeLog:
PR fortran/100018
* resolve.c: Add association check before de-referencing
pointer.
gcc/testsuite/ChangeLog:
PR fortran/100018
* gfortran.dg/PR10018.f90: New test.
Thank you very much.
Best regards,
José Rui
-----------------
Mentor Graphics (Deutschland) GmbH, Arnulfstrasse 201, 80634 München
Registergericht München HRB 106955, Geschäftsführer: Thomas Heurung, Frank
Thürauf
