[clang-tools-extra] 5be4536 - [clang-reorder-fields] Avoid repeated hash lookups (NFC) (#127165)
via cfe-commits
cfe-commits at lists.llvm.org
Fri Feb 14 01:33:22 PST 2025
Author: Kazu Hirata
Date: 2025-02-14T01:33:17-08:00
New Revision: 5be4536d091d7b927ceaa30a263ab5629f9008cc
URL: https://github.com/llvm/llvm-project/commit/5be4536d091d7b927ceaa30a263ab5629f9008cc
DIFF: https://github.com/llvm/llvm-project/commit/5be4536d091d7b927ceaa30a263ab5629f9008cc.diff
LOG: [clang-reorder-fields] Avoid repeated hash lookups (NFC) (#127165)
Added:
Modified:
clang-tools-extra/clang-reorder-fields/ReorderFieldsAction.cpp
Removed:
################################################################################
diff --git a/clang-tools-extra/clang-reorder-fields/ReorderFieldsAction.cpp b/clang-tools-extra/clang-reorder-fields/ReorderFieldsAction.cpp
index aeb7fe90f2175..ea0207619fb2b 100644
--- a/clang-tools-extra/clang-reorder-fields/ReorderFieldsAction.cpp
+++ b/clang-tools-extra/clang-reorder-fields/ReorderFieldsAction.cpp
@@ -70,11 +70,12 @@ getNewFieldsOrder(const RecordDecl *Definition,
}
SmallVector<unsigned, 4> NewFieldsOrder;
for (const auto &Name : DesiredFieldsOrder) {
- if (!NameToIndex.count(Name)) {
+ auto It = NameToIndex.find(Name);
+ if (It == NameToIndex.end()) {
llvm::errs() << "Field " << Name << " not found in definition.\n";
return {};
}
- NewFieldsOrder.push_back(NameToIndex[Name]);
+ NewFieldsOrder.push_back(It->second);
}
assert(NewFieldsOrder.size() == NameToIndex.size());
return NewFieldsOrder;
More information about the cfe-commits
mailing list