Skip to content

Conversation

kazutakahirata
Copy link
Contributor

Both Size and Count are just integers, so I am not using & here.

Both Size and Count are just integers, so I am not using & here.
@llvmbot
Copy link
Member

llvmbot commented Oct 2, 2025

@llvm/pr-subscribers-llvm-adt

Author: Kazu Hirata (kazutakahirata)

Changes

Both Size and Count are just integers, so I am not using & here.


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

1 Files Affected:

  • (modified) llvm/include/llvm/ADT/ConcurrentHashtable.h (+2-3)
diff --git a/llvm/include/llvm/ADT/ConcurrentHashtable.h b/llvm/include/llvm/ADT/ConcurrentHashtable.h
index 6de194db9ba7a..6a943c5b062e7 100644
--- a/llvm/include/llvm/ADT/ConcurrentHashtable.h
+++ b/llvm/include/llvm/ADT/ConcurrentHashtable.h
@@ -253,9 +253,8 @@ class ConcurrentHashTableByPtr {
 
     OS << "\nOverall number of entries = " << OverallNumberOfEntries;
     OS << "\nOverall number of non empty buckets = " << NumberOfNonEmptyBuckets;
-    for (auto &BucketSize : BucketSizesMap)
-      OS << "\n Number of buckets with size " << BucketSize.first << ": "
-         << BucketSize.second;
+    for (auto [Size, Count] : BucketSizesMap)
+      OS << "\n Number of buckets with size " << Size << ": " << Count;
 
     std::stringstream stream;
     stream << std::fixed << std::setprecision(2)

@kazutakahirata kazutakahirata merged commit 0dd8f32 into llvm:main Oct 2, 2025
11 checks passed
@kazutakahirata kazutakahirata deleted the cleanup_20251001_ADT_ConcurrentHashtable branch October 2, 2025 16:28
mahesh-attarde pushed a commit to mahesh-attarde/llvm-project that referenced this pull request Oct 3, 2025
Both Size and Count are just integers, so I am not using & here.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants