[PATCH] D84195: [flang][openacc] Skeleton for OpenACC construct lowering

[Brian Yang via Phabricator via llvm-commits]

ichoyjx added inline comments.


================
Comment at: flang/include/flang/Lower/PFTBuilder.h:135
+    std::tuple<parser::CompilerDirective, parser::OpenACCConstruct,
+               parser::OpenMPConstruct, parser::OmpEndLoopDirective>;
 
----------------
We may need `EndLoopDirective` for OpenACC also because for combined constructs, the `end * loop` is optional.


================
Comment at: flang/test/Lower/pre-fir-tree05.f90:29
+  ! CHECK-NEXT: <<End OpenACCConstruct>>
+  ! CHECK-NEXT: <<End OpenACCConstruct>>
+  ! CHECK-NEXT: ContinueStmt
----------------
What is the second `<<End OpenACCConstruct>>` here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84195