Skip to content

Conversation

@mtrofin
Copy link
Member

@mtrofin mtrofin commented Aug 15, 2025

No description provided.

Copy link
Member Author

mtrofin commented Aug 15, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@mtrofin mtrofin marked this pull request as ready for review August 15, 2025 06:22
@llvmbot
Copy link
Member

llvmbot commented Aug 15, 2025

@llvm/pr-subscribers-llvm-ir

Author: Mircea Trofin (mtrofin)

Changes

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

1 Files Affected:

  • (modified) llvm/lib/IR/ProfDataUtils.cpp (+1-1)
diff --git a/llvm/lib/IR/ProfDataUtils.cpp b/llvm/lib/IR/ProfDataUtils.cpp
index c90a1efb9a54a..d24263f8b3bda 100644
--- a/llvm/lib/IR/ProfDataUtils.cpp
+++ b/llvm/lib/IR/ProfDataUtils.cpp
@@ -276,7 +276,7 @@ SmallVector<uint32_t> downscaleWeights(ArrayRef<uint64_t> Weights,
                                                 : *llvm::max_element(Weights);
   assert(MaxCount > 0 && "Bad max count");
   uint64_t Scale = calculateCountScale(MaxCount);
-  SmallVector<unsigned> DownscaledWeights;
+  SmallVector<uint32_t> DownscaledWeights;
   for (const auto &ECI : Weights)
     DownscaledWeights.push_back(scaleBranchCount(ECI, Scale));
   return DownscaledWeights;

@mtrofin mtrofin merged commit 45e6951 into main Aug 15, 2025
11 of 13 checks passed
@mtrofin mtrofin deleted the users/mtrofin/08-14-use_uint32_t_rather_than_unsigned_in_downscaleweights_ branch August 15, 2025 06:22
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.

3 participants