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

Reply via email to