[PATCH] D93205: [Flang][OpenMP 4.5] Add semantic check for OpenMP Do Loop Constructs for single directive

[Kiran Chandramohan via Phabricator via llvm-commits]

kiranchandramohan added a comment.

Why are you changing the tests while removing XFAIL? eg. omp-do05.f90 was supposed to check chunk_size.



================
Comment at: flang/lib/Semantics/check-omp-structure.cpp:518
+
+void OmpStructureChecker::Enter(const parser::OmpClause::Firstprivate &x) {
+  CheckAllowed(llvm::omp::Clause::OMPC_firstprivate);
----------------
We capture data-sharing attributes in resolve-directives right? Why is this function better here compared to resolve-directives?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93205