ABataev added inline comments.

================
Comment at: lib/Basic/OpenMPKinds.cpp:685
 
 bool clang::isOpenMPWorksharingDirective(OpenMPDirectiveKind DKind) {
   return DKind == OMPD_for || DKind == OMPD_for_simd ||
----------------
I think this new directive must be considered as a worksharing directive also.


================
Comment at: lib/Parse/ParseOpenMP.cpp:21
 #include "llvm/ADT/PointerIntPair.h"
-
 using namespace clang;
----------------
Restore this line


================
Comment at: test/OpenMP/nesting_of_regions.cpp:3326
   }
-#pragma omp ordered
   {
----------------
what about teams distribute parallel for simd inside the ordered directive? Why 
this one is removed?


https://reviews.llvm.org/D27084



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

Reply via email to