[clang][NFC] Don't copy into a vector just to iterate in IsInitListMemberExprInitialized

[davidstone]

No description provided.

[llvmbot]

@llvm/pr-subscribers-clang

Author: David Stone (davidstone)

Changes

---

Full diff: https://github.com/llvm/llvm-project/pull/169385.diff

1 Files Affected:

• (modified) clang/lib/Sema/SemaDeclCXX.cpp (+11-11)

diff --git a/clang/lib/Sema/SemaDeclCXX.cpp b/clang/lib/Sema/SemaDeclCXX.cpp index 8030aac3d8771..3bc748969065a 100644 --- a/clang/lib/Sema/SemaDeclCXX.cpp +++ b/clang/lib/Sema/SemaDeclCXX.cpp @@ -3782,21 +3782,21 @@ namespace { if (CheckReferenceOnly && !ReferenceField) return true; - llvm::SmallVector<unsigned, 4> UsedFieldIndex; // Discard the first field since it is the field decl that is being // initialized. - for (const FieldDecl *FD : llvm::drop_begin(llvm::reverse(Fields))) - UsedFieldIndex.push_back(FD->getFieldIndex()); - - for (auto UsedIter = UsedFieldIndex.begin(), - UsedEnd = UsedFieldIndex.end(), - OrigIter = InitFieldIndex.begin(), - OrigEnd = InitFieldIndex.end(); - UsedIter != UsedEnd && OrigIter != OrigEnd; ++UsedIter, ++OrigIter) { - if (*UsedIter < *OrigIter) + auto UsedFields = llvm::drop_begin(llvm::reverse(Fields)); + auto UsedIter = UsedFields.begin(); + const auto UsedEnd = UsedFields.end(); + + for (const unsigned Orig : InitFieldIndex) { + if (UsedIter == UsedEnd) + break; + const unsigned UsedIndex = (*UsedIter)->getFieldIndex(); + if (UsedIndex < Orig) return true; - if (*UsedIter > *OrigIter) + if (UsedIndex > Orig) break; + ++UsedIter; } return false;