[flang-commits] [PATCH] D89879: [Flang][OpenMP] Parsing and semantic changes for flush construct from OMP 5.0 specification.

[sameeran joshi via flang-commits]

I usually don't build clang.
I need to try.


On Wed 21 Oct, 2020, 11:59 PM Valentin Clement via Phabricator, <
reviews at reviews.llvm.org> wrote:

> clementval added inline comments.
>
>
> ================
> Comment at: llvm/include/llvm/Frontend/OpenMP/OMP.td:423
>  def OMP_Flush : Directive<"flush"> {
> -  let allowedClauses = [
> +  let allowedOnceClauses = [
>      VersionedClause<OMPC_AcqRel, 50>,
> ----------------
> Did you check if clang test are still passing with this change?
>
>
> Repository:
>   rG LLVM Github Monorepo
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D89879/new/
>
> https://reviews.llvm.org/D89879
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/flang-commits/attachments/20201022/182c5720/attachment.html>