[PATCH] D91839: [flang][openmp] Separate memory-order-clause parser creating OmpClause node
Kiran Chandramohan via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Nov 21 05:29:12 PST 2020
kiranchandramohan accepted this revision.
kiranchandramohan added a comment.
> @kiranchandramohan does this seem ok from query which was raised in https://reviews.llvm.org/D89583#inline-836220 comment ?
Thanks @clementval, @sameeranjoshi LGTM.
================
Comment at: flang/lib/Parser/openmp-parsers.cpp:304
-// 2.17.8 Flush construct [OpenMP 5.0]
-// flush -> FLUSH [memory-order-clause] [(variable-name-list)]
-// memory-order-clause -> acq_rel
+// 2.17.7 Atomtic construct/2.17.8 Flush construct [OpenMP 5.0]
+// memory-order-clause ->
----------------
Nit: Atomtic -> Atomic
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91839/new/
https://reviews.llvm.org/D91839
More information about the llvm-commits
mailing list