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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
