[PATCH] D106335: [flang][OpenMP] Add semantic check for teams nesting

Peixin Qiao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 20 23:24:42 PDT 2021


peixin added inline comments.


================
Comment at: flang/test/Semantics/omp-firstprivate01.f90:15
 
+  !ERROR: TARGET construct with nested TEAMS region contains statements outside of the TEAMS construct
   !$omp target
----------------
clementval wrote:
> Which statement is outside? 
This target directive has two teams construct nested inside and both of teams construct contain statements. Change the error info by replacing `statements` by `statements or directives`, which should be more reasonable.


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

https://reviews.llvm.org/D106335



More information about the llvm-commits mailing list