[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