[flang-commits] [PATCH] D114181: Add semantics unit test for SYNC ALL statement

[Andrzej Warzynski via Phabricator via flang-commits]

awarzynski added a comment.
Herald added a project: All.

Thanks for working on this, @rouson !

I've just noticed #53915 <https://github.com/llvm/llvm-project/issues/53915> - thank you for flagging this! LLVM Flang has only recently gained F95 <https://reviews.llvm.org/F95> support and I imagine it might be a while (perhaps years) before F18 <https://reviews.llvm.org/F18> support is available. It is possible that we won't be re-visiting this in the meantime and once we do come back to this we might find the rationale behind `XFAIL`ing this test rather unclear (or hard to track). I think that it would be worth adding a comment that there's no F18 <https://reviews.llvm.org/F18> support available yet and hence this test can neither be run nor tested.

Thank you,
-Andrzej


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114181