ABataev added inline comments.

================
Comment at: lib/Sema/SemaOpenMP.cpp:343
+    for (unsigned I = Stack.size() - 1; I > 0; --I)
+      if (checkMappableExprComponentListsForDeclAtLevel(VD, I - 2, Check))
+        return true;
----------------
Are you sure this is correct? If I is 1, I-2 will give you 0xFFFFFFFF. Please, 
check everything one more time


================
Comment at: lib/Sema/SemaOpenMP.cpp:355
+               OpenMPClauseKind)> &Check) {
+    // auto StartI = std::next(Stack.begin());
+    auto StartI = Stack.begin();
----------------
Please, remove this commented code


Repository:
  rL LLVM

https://reviews.llvm.org/D29905



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

Reply via email to