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

Valentin Clement via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 20 08:08:22 PST 2020


clementval added inline comments.


================
Comment at: flang/include/flang/Parser/parse-tree.h:3625
+  std::tuple<Verbatim, OmpObjectList, OmpClauseList> t;
+};
+
----------------
Is there a good reason to have separate nodes (`OpenMPDeclarativeAllocate`, `OpenMPExecutableAllocateList`)? Both node have the same content. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89562



More information about the llvm-commits mailing list