ABataev added inline comments.

================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:6230
     BindKind = BC->getBindKind();
+
   // First check CancelRegion which is then used in checkNestingOfRegions.
----------------
Remove this new line


================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:6245-6248
+  UseClausesWithoutBind = mapLoopConstruct(
+      ClausesWithoutBind, Clauses, BindKind, Kind, PrevMappedDirective,
+      StartLoc, EndLoc, DirName, CancelRegion);
 
----------------
Can we do this before checks in lines 6232-6235 so we can avoid double call of 
checkNestingOfRegions for bind?


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

https://reviews.llvm.org/D158266

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

Reply via email to