[PATCH] D94087: [flang][openmp]At most one threads, simd and depend clause can appear on OpenMP ORDERED construct.

Valentin Clement via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 22 07:45:50 PST 2021


clementval added inline comments.


================
Comment at: flang/lib/Semantics/check-omp-structure.cpp:891
+  int dependSourceCount{0};
+  for (const auto &clause : ompClauseList.v) {
+    if (const auto *dependClause{
----------------
What is the final reason to not use `FindClause`? It would be more efficient than iterate over the clause list once more. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94087



More information about the llvm-commits mailing list