[llvm] [LLVM][DWARF] Make dwarf::getDebugNamesBucketCount return a pair. (PR #83047)

via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 26 13:04:04 PST 2024


================
@@ -616,20 +616,20 @@ enum AcceleratorTable {
 // Uniquify the string hashes and calculate the bucket count for the
 // DWARF v5 Accelerator Table. NOTE: This function effectively consumes the
 // 'hashes' input parameter.
-inline uint32_t getDebugNamesBucketCount(MutableArrayRef<uint32_t> hashes,
-                                         uint32_t &uniqueHashCount) {
-  uint32_t BucketCount = 0;
+inline std::pair<uint32_t, uint32_t>
+getDebugNamesBucketAndHashCount(MutableArrayRef<uint32_t> hashes) {
+  uint32_t bucketCount = 0;
----------------
cmtice wrote:

I thought LLVM variables were supposed to start with lower case?  So I thought I was fixing a problem? I can switch it back if you think I should...

https://github.com/llvm/llvm-project/pull/83047


More information about the llvm-commits mailing list