[llvm] [Transforms] Migrate to a new version of getValueProfDataFromInst (PR #95477)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 13 14:37:30 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-pgo
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
Note that the version of getValueProfDataFromInst that returns bool
has been "deprecated" since:
commit 1e15371dd8843dfc52b9435afaa133997c1773d8
Author: Mingming Liu <mingmingl@<!-- -->google.com>
Date: Mon Apr 1 15:14:49 2024 -0700
---
Full diff: https://github.com/llvm/llvm-project/pull/95477.diff
1 Files Affected:
- (modified) llvm/lib/Transforms/Instrumentation/CGProfile.cpp (+5-5)
``````````diff
diff --git a/llvm/lib/Transforms/Instrumentation/CGProfile.cpp b/llvm/lib/Transforms/Instrumentation/CGProfile.cpp
index c322d0abd6bc1..651239bee91f9 100644
--- a/llvm/lib/Transforms/Instrumentation/CGProfile.cpp
+++ b/llvm/lib/Transforms/Instrumentation/CGProfile.cpp
@@ -78,14 +78,14 @@ static bool runCGProfilePass(Module &M, FunctionAnalysisManager &FAM,
if (!CB)
continue;
if (CB->isIndirectCall()) {
- InstrProfValueData ValueData[8];
uint32_t ActualNumValueData;
uint64_t TotalC;
- if (!getValueProfDataFromInst(*CB, IPVK_IndirectCallTarget, 8,
- ValueData, ActualNumValueData, TotalC))
+ auto ValueData = getValueProfDataFromInst(
+ *CB, IPVK_IndirectCallTarget, 8, ActualNumValueData, TotalC);
+ if (!ValueData)
continue;
- for (const auto &VD :
- ArrayRef<InstrProfValueData>(ValueData, ActualNumValueData)) {
+ for (const auto &VD : ArrayRef<InstrProfValueData>(
+ ValueData.get(), ActualNumValueData)) {
UpdateCounts(TTI, &F, Symtab.getFunction(VD.Value), VD.Count);
}
continue;
``````````
</details>
https://github.com/llvm/llvm-project/pull/95477
More information about the llvm-commits
mailing list