Dear Alessandro,
Alessandro Fanfarillo wrote:
the compiler returns an ICE when a coarray critical section is used
inside a module procedure.
The symbols related with the lock variables were left uncommitted
inside resolve_critical(). A gfc_commit_symbol after each symbol or a
gfc_commit_symbols at the end of resolve_critical() fixed the issue.
The latter solution is proposed in the attached patch.
Built and regtested on x86_64-pc-linux-gnu
Looks good to me.
PS: This patch should be also included in GCC 5.
Yes, that's fine with me.
Tobias
PS: I saw that you now have a GCC account, which you can use to commit
to both the trunk and gcc-5-branch. See https://gcc.gnu.org/svnwrite.html.
Additionally, you should update MAINTAINERS (trunk only) by adding
yourself under "Write After Approval"; you can simply commit this patch
yourself, but you should write an email to gcc-patches with the patch -
like Alan did at https://gcc.gnu.org/ml/gcc-patches/2015-10/msg02390.html