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

Valentin Clement via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 27 08:43:54 PDT 2020


clementval added inline comments.


================
Comment at: flang/include/flang/Parser/dump-parse-tree.h:553
+  NODE(parser, OpenMPDeclarativeAllocate)
+  /*NODE(OpenMPDeclarativeAllocate, AllocatorClause)*/
   NODE(parser, OpenMPDeclarativeConstruct)
----------------
Any reason to keep this? 


================
Comment at: flang/include/flang/Parser/dump-parse-tree.h:562
+  NODE(parser, OpenMPExecutableAllocate)
+  /*NODE(OpenMPExecutableAllocate, AllocatorClause)*/
   NODE(parser, OpenMPSimpleStandaloneConstruct)
----------------
Any reason to keep this? 


================
Comment at: flang/test/Parser/omp-allocate-unparse.f90:27
+!CHECK: !$OMP ALLOCATE
\ No newline at end of file

----------------
Newline. 


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