[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