[PATCH] D85212: [flang] Add parser support for OpenMP allocate clause

Irina Dobrescu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 6 03:06:10 PDT 2020


Rin marked an inline comment as done.
Rin added a comment.

In D85212#2196478 <https://reviews.llvm.org/D85212#2196478>, @sscalpone wrote:

> Do you have a negative test that checks for an out-of-place allocate?

Not yet, but they will be added in the Semantic checks, which I'll work on next.



================
Comment at: flang/include/flang/Parser/parse-tree.h:3498
       OmpIfClause, OmpLinearClause, OmpMapClause, OmpProcBindClause,
-      OmpReductionClause, OmpScheduleClause>
+      OmpReductionClause, OmpAllocateClause, OmpScheduleClause>
       u;
----------------
kiranchandramohan wrote:
> Are the Omp*Clauses in alphabetic order?
Yeah, you're right, I'll change that


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85212



More information about the llvm-commits mailing list