[PATCH] D137851: [OPENMP]Initial support for at clause

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 11 09:35:54 PST 2022


ABataev added inline comments.


================
Comment at: clang/include/clang/AST/OpenMPClause.h:1613-1615
+  /// Sets the location of '('.
+  void setLParenLoc(SourceLocation Loc) { LParenLoc = Loc; }
+
----------------
Make it private


================
Comment at: clang/include/clang/Parse/Parser.h:3296-3304
+  /// Parses clauses for directive.
+  ///
+  /// \param DKind Kind of current directive.
+  /// \param clauses for current directive.
+  /// \param start location for clauses of current directive
+  void ParseOpenMPClauses(OpenMPDirectiveKind DKind,
+                          SmallVector<clang::OMPClause *, 1> *Clauses,
----------------
Could you make it in a separate NFC patch?


================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:11034-11037
+  for (const OMPClause *CNew : Clauses) {
+    if (auto *AtCl = cast<OMPAtClause>(CNew))
+      AtC = AtCl;
+  }
----------------
`find_first_of`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137851



More information about the llvm-commits mailing list