Re: [Patch, fortran] PR88632 (& 104630) - [F08] function contained in module invisible to submodule unless declared public

Hi Paul!

Am 01.06.26 um 2:03 PM schrieb Paul Richard Thomas:
> Hello All,
> 
> The attached patch is relatively trivial. It regtests OK on Fedora 44/x86_64.
> 
> OK for mainline and for backporting to 16-branch?

Ehm, the first hunk

--- a/gcc/fortran/module.cc
+++ b/gcc/fortran/module.cc
@@ -4372,7 +4372,7 @@ mio_full_f2k_derived (gfc_symbol *sym)
  	  /* PDT templates make use of the mechanisms for formal args
  	     and so the parameter symbols are stored in the formal
  	     namespace.  Transfer the sym_root to f2k_derived and then
-	     free the formal namespace since it is unneeded.  */
+	     free the formal namespace since it is uneeded.  */
  	  if (sym->attr.pdt_template && sym->formal && sym->formal->sym)
  	    {
  	      ns = sym->formal->sym->ns;

seems incorrect to me.  And my mail reader agrees with me...

The rest of the patch LGTM.

Thanks for the patch!

Harald

> Regards
> 
> Paul