Hi Paul, all,
 
having really enjoyed the review process, I've now committed Paul's version
including his comment.  See also attached.

Thanks,
Harald

 
Gesendet: Freitag, 16. April 2021 um 13:02 Uhr
Von: "Paul Richard Thomas" <paul.richard.thomas@gmail.com>
An: "Bernhard Reutner-Fischer" <rep.dot.nop@gmail.com>
Cc: "Harald Anlauf via Fortran" <fortran@gcc.gnu.org>, "Harald Anlauf" <anlauf@gmx.de>
Betreff: Re: [PATCH] PR fortran/63797 - Bogus ambiguous reference to 'sqrt'

Hi Harald,
 
This is the output from a completely tedious Zoom meeting:
 
  /* From F2003 onwards, intrinsic procedures are no longer subject to
     the restriction, "that an elemental intrinsic function here be of
     type integer or character and each argument must be an initialization
     expr of type integer or character" is lifted so that intrinsic
     procedures can be over-ridden. This requires that the intrinsic
     symbol not appear in the module file, thereby preventing ambiguity
     when USEd.  */
  if (strcmp (sym->module, "(intrinsic)") == 0
      && (gfc_option.allow_std & GFC_STD_F2003))
    return;
 
Modify or replace the comment, as desired. The change to the condition gives the desired result in terms of standard compliance.
 
Regards
 
Paul


PR fortran/63797 - Bogus ambiguous reference to 'sqrt'

The interface of an intrinsic procedure is automatically explicit.
Do not write it to the module file to prevent wrong ambiguities on USE.

gcc/fortran/ChangeLog:

	PR fortran/63797
	* module.c (write_symtree): Do not write interface of intrinsic
	procedure to module file for F2003 and newer.

gcc/testsuite/ChangeLog:

	PR fortran/63797
	* gfortran.dg/pr63797.f90: New test.

Co-authored-by: Paul Thomas <pault@gcc.gnu.org>