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

Valentin Clement via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 17 11:27:45 PDT 2020


clementval marked an inline comment as done.
clementval added inline comments.


================
Comment at: flang/lib/Semantics/resolve-names.cpp:1085
 
+template <typename T> class DirectiveAttributeVisitor {
+public:
----------------
klausler wrote:
> resolve-names.cpp is already a large source file.  Does the directive name resolution code have to be in there in order to use things that are local to that file?  If not, maybe all the directive name resolution code could reside elsewhere.
I think we can move that into a separate file. I'll update the patch. Thanks for the suggestion.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83998





More information about the llvm-commits mailing list