[flang-commits] [PATCH] D137204: [flang] Add check for constraints on event-stmts

Pete Steinfeld via Phabricator via flang-commits flang-commits at lists.llvm.org
Mon May 15 11:08:51 PDT 2023


PeteSteinfeld added a comment.

In D137204#4342812 <https://reviews.llvm.org/D137204#4342812>, @ktras wrote:

> Unfortunately, I let this patch sitting for a while, but I would like to push this soon. @PeteSteinfeld, thank you for the review and for the acceptance of the patch. Previously, I replied to your comment about moving `IsEventType` and `IsLockType` and stating that because of future planned changes to support `event_query`, it seems to me that these new functions should remain where they are now in this patch. Please let me know if you disagree, otherwise I plan to push this differential within the next couple days based on the revision having been accepted. Let me know if you would like me to hold off for further discussion, thanks.

Thanks, @ktras.  It's OK with me to leave them where they are.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137204/new/

https://reviews.llvm.org/D137204



More information about the flang-commits mailing list