[llvm] [SampleProfileLoader] Fix integer overflow in generateMDProfMetadata (PR #90217)
Nabeel Omer via llvm-commits
llvm-commits at lists.llvm.org
Wed May 8 06:23:17 PDT 2024
================
@@ -1713,13 +1713,15 @@ void SampleProfileLoader::generateMDProfMetadata(Function &F) {
// if needed. Sample counts in profiles are 64-bit unsigned values,
// but internally branch weights are expressed as 32-bit values.
if (Weight > std::numeric_limits<uint32_t>::max()) {
- LLVM_DEBUG(dbgs() << " (saturated due to uint32_t overflow)");
+ LLVM_DEBUG(dbgs() << " (saturated due to uint32_t overflow)\n");
Weight = std::numeric_limits<uint32_t>::max();
}
if (!SampleProfileUseProfi) {
// Weight is added by one to avoid propagation errors introduced by
// 0 weights.
- Weights.push_back(static_cast<uint32_t>(Weight + 1));
+ Weights.push_back(static_cast<uint32_t>(
----------------
omern1 wrote:
Thanks! I'll look into this soon and share the results.
https://github.com/llvm/llvm-project/pull/90217
More information about the llvm-commits
mailing list