[clang] [clang][NFC] Don't copy into a vector just to iterate in `IsInitListMemberExprInitialized` (PR #169385)
David Stone via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 24 10:31:42 PST 2025
https://github.com/davidstone created https://github.com/llvm/llvm-project/pull/169385
None
>From 1c923448e4a4618b67c51d6fedd80275bf5d0e0e Mon Sep 17 00:00:00 2001
From: David Stone <davidfromonline at gmail.com>
Date: Mon, 24 Nov 2025 11:30:51 -0700
Subject: [PATCH] [clang][NFC] Don't copy into a vector just to iterate in
`IsInitListMemberExprInitialized`.
---
clang/lib/Sema/SemaDeclCXX.cpp | 22 +++++++++++-----------
1 file changed, 11 insertions(+), 11 deletions(-)
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;
More information about the cfe-commits
mailing list