[Patch, fortran] PR66549 [5/6 regression] ICE with OMP PARALLEL

[Jakub Jelinek]

On Thu, Jun 18, 2015 at 08:26:19PM +0200, Mikael Morin wrote:
> Hmm, for gfc_derived_types at least, I would say no.
> gfc_derived_types seems to be a per root namespace global, in other
> words shared between contained procedures, so it should not be
> saved/restored upon resolution of sibling or internal procedures.
> And I don't think it's possible to escape one root namespace;
> use-associated symbols are not shared with the corresponding module
> symbols for example.
> So we are (quite) safe here, I think.

Ok.

> To be honest, I'm not at ease with these globals,  both openmp and
> derived_types, and I'm not fond of starting resolution of a different
> procedure in the middle of nowhere.  I can try to produce a patch that
> walks the procedure code to pick the needed information instead of doing
> full procedure resolutions at arbitrary places, if that seems worth it.
>  Or try to remove gfc_derived_types.  But IMHO the patch proposed here
> makes sense in any case.

If the patch passes regtest (both check-gfortran and libgomp fortran.exp),
then it is ok for trunk/5 branch.

	Jakub