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

[Tim Keith via Phabricator via llvm-commits]

tskeith accepted this revision.
tskeith added inline comments.


================
Comment at: flang/lib/Semantics/resolve-names.cpp:6488
   inExecutionPart_ = false;
+  ResolveAccParts(x);
   ResolveOmpParts(x);
----------------
clementval wrote:
> tskeith wrote:
> > Is it worthwhile skipping this unless `-fopenacc` is present? (Same with OpenMP?)
> Would make sense I guess. Is there an easy way to query which options are enabled from here? 
Use `SemanticsContext::IsEnabled` and pass in the `LanguageFeature` for OpenACC or OpenMP.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83998