[PATCH] D83998: [flang][openacc] Basic name resolution infrastructure for OpenACC construct

[Valentin Clement via Phabricator via llvm-commits]

clementval marked 2 inline comments as done.
clementval added inline comments.


================
Comment at: flang/lib/Semantics/resolve-names.cpp:1293
+  bool Pre(const parser::SpecificationPart &x) {
+    Walk(std::get<std::list<parser::OpenMPDeclarativeConstruct>>(x.t));
+    return false;
----------------
ichoyjx wrote:
> I believe this is a typo: `OpenMP`.
Good catch! Just updated the diff. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83998