From 6bde64064f1b8b5178c9ab8edf5d7a6e34c43966 Mon Sep 17 00:00:00 2001 From: Kazu Hirata Date: Sat, 17 May 2025 10:37:39 -0700 Subject: [PATCH] [clang] Use DenseMap::keys (NFC) With DenseMap::keys, we don't need to use [[maybe_unused]]. --- clang/include/clang/Analysis/FlowSensitive/StorageLocation.h | 2 +- clang/lib/Analysis/FlowSensitive/ASTOps.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/clang/include/clang/Analysis/FlowSensitive/StorageLocation.h b/clang/include/clang/Analysis/FlowSensitive/StorageLocation.h index 5d9a0f702a299..8948428accb97 100644 --- a/clang/include/clang/Analysis/FlowSensitive/StorageLocation.h +++ b/clang/include/clang/Analysis/FlowSensitive/StorageLocation.h @@ -131,7 +131,7 @@ class RecordStorageLocation final : public StorageLocation { << " on StorageLocation " << this << " of type " << getType() << "\n"; llvm::dbgs() << "Existing children:\n"; - for ([[maybe_unused]] auto [Field, Loc] : Children) { + for (const auto &Field : Children.keys()) { llvm::dbgs() << Field->getNameAsString() << "\n"; } } diff --git a/clang/lib/Analysis/FlowSensitive/ASTOps.cpp b/clang/lib/Analysis/FlowSensitive/ASTOps.cpp index 431b1f2038357..7ce6b03fc0e71 100644 --- a/clang/lib/Analysis/FlowSensitive/ASTOps.cpp +++ b/clang/lib/Analysis/FlowSensitive/ASTOps.cpp @@ -81,7 +81,7 @@ bool containsSameFields(const FieldSet &Fields, const RecordStorageLocation::FieldToLoc &FieldLocs) { if (Fields.size() != FieldLocs.size()) return false; - for ([[maybe_unused]] auto [Field, Loc] : FieldLocs) + for (const auto &Field : FieldLocs.keys()) if (!Fields.contains(cast_or_null(Field))) return false; return true;