[flang-commits] [flang] [llvm] [flang][MIF] Adding lowering for EVENT POST/WAIT statements (PR #210283)

[via flang-commits]

MattPD wrote:

@rouson Thanks, I agree in principle, and I definitely prefer to find everything in one go. At the same time, I hope it's useful to surface potentially serious findings (crashes, miscompilations, memory-safety issues) before the merge, assuming it's better we fix these before they reach end users.

Of the remaining issues, I'd consider only three to be blockers: The Flang runtime still frees the storage of a VOLATILE character array coarray on a whole-array assignment; the compiler now crashes on some assignments to allocatable or CONTIGUOUS array coarrays; and a missing include in `MIFOpConversion.cpp` now breaks the CI build. The first two involve the coarray assignment fix in the latest commit.

My latest comments suggest one small change per problem. Each suggested change fixed the corresponding failure in my local build.

@JDPailleux, a quick reminder: Please update the PR description before the merge, as I asked in my [first review](https://github.com/llvm/llvm-project/pull/210283#pullrequestreview-5446356783).

https://github.com/llvm/llvm-project/pull/210283