Minor comment change, use llvm::enumerage

Author: abhinavgaba

clang/lib/AST/OpenMPClause.cpp

@@ -1202,8 +1202,12 @@ OMPClauseMappableExprCommon::findAttachPtrExpr(
   // stripping away one component at a time, until we reach a pointer Expr
   // (that is not a binary operator). The first such pointer should be the
   // attach base-pointer for the component list.
-  for (size_t I = 1; I < Components.size(); ++I) {
-    const Expr *CurExpr = Components[I].getAssociatedExpression();
+  for (auto [I, Component] : llvm::enumerate(Components)) {
+    // Skip past the first component.
+    if (I == 0)
+      continue;
+
+    const Expr *CurExpr = Component.getAssociatedExpression();
     if (!CurExpr)
       break;
 

clang/lib/CodeGen/CGOpenMPRuntime.cpp

@@ -7209,8 +7209,8 @@ class MappableExprsHandler {
   /// The component-depth of `nullptr` (i.e. no attach-ptr) is `std::nullopt`.
   /// TODO: Not urgent, but we should ideally use the number of pointer
   /// dereferences in an expr as an indicator of its complexity, instead of the
-  /// component-depth. That would be needed for us to treat `p[1]`, `*(p + 10)`,
-  /// `*(p + 5 + 5)` together.
+  /// component-depth. That would be needed for us to treat `p[10]`,
+  /// `*(p + 10)`, `*(p + 5 + 5)` together.
   mutable llvm::DenseMap<const Expr *, std::optional<size_t>>
       AttachPtrComponentDepthMap = {{nullptr, std::nullopt}};