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

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added inline comments.


================
Comment at: clang/include/clang/Sema/Sema.h:11293
+                                       SourceLocation EndLoc,
+                                       bool InExContext = true);
   /// Called on well-formed '\#pragma omp barrier'.
----------------
Why do you need this flag?


================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:11035-11037
+  for (auto *AC :
+       OMPExecutableDirective::getClausesOfKind<OMPAtClause>(Clauses))
+    AtC = AC;
----------------
Why do you need a loop here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137851