ABataev added inline comments.

================
Comment at: clang/include/clang/AST/StmtOpenMP.h:373
+///
+class OMPMetaDirective final : public OMPExecutableDirective {
+  friend class ASTStmtReader;
----------------
I think, metadirective should be a kind of a container for different 
sub-directives. The problem is that that subdirectives could be completely 
different, they may capture different variables, using different capture kind 
(by value or by reference) etc.So, you need to generate each possible 
sub-directive independently and store them in the meta directive node. 
Otherwise you won't be able to generate the code correctly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91944

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to