[PATCH] D106896: [flang][OpenMP] Add parsing support for nontemporal clause.

Arnamoy B via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 27 13:29:31 PDT 2021


arnamoy10 added inline comments.


================
Comment at: flang/include/flang/Parser/parse-tree.h:3380
+// nontemporal-clause -> NONTEMPORAL (variable-name-list)
+struct OmpNontemporalClause {
+  TUPLE_CLASS_BOILERPLATE(OmpNontemporalClause);
----------------
clementval wrote:
> Non necessary with update to `OMP.td`
Thanks for the comments.  However, are you sure about the removal of this struct definition?  Because now from `flang/lib/Parser/openmp-parsers.cpp`, I am getting the following error:

```
flang/lib/Parser/openmp-parsers.cpp:153:23: error: ‘OmpNontemporalClause’ was not declared in this scope 
153 | TYPE_PARSER(construct<OmpNontemporalClause>(nonemptyList(name)))
```
Thanks for the help.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106896



More information about the llvm-commits mailing list