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

David Truby via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 4 08:19:31 PDT 2020


DavidTruby requested changes to this revision.
DavidTruby added a comment.
This revision now requires changes to proceed.

LGTM other than formatting, could you run clang-format over this please? Thanks!



================
Comment at: flang/lib/Parser/openmp-parsers.cpp:109
+TYPE_PARSER(construct<OmpAllocateClause>(
+    maybe(construct<OmpAllocateClause::Allocator>(scalarIntExpr) / ":"), Parser<OmpObjectList>{}))
+
----------------
I think this line would get split in 2 by clang-format as it is over 80 characters


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