[PATCH] D89562: [flang]Add Parser Support for OpenMP Allocate Directive

[Valentin Clement via Phabricator via llvm-commits]

clementval accepted this revision.
clementval added a comment.

Just have a small comment. Otherwise I think it looks good to me. Please wait for the approval of someone on the OpenMP side.



================
Comment at: flang/lib/Semantics/check-omp-structure.h:145
   void Enter(const parser::OmpClause::NumThreads &);
+  void Enter(const parser::OmpClause::Allocator &);
   void Enter(const parser::OmpClause::Ordered &);
----------------
Can you move this up. I think the enter clause function are listed by alphabetical order. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89562