Skip to content

Commit

Permalink
[llvm-profdata] Clean up traverseAllValueSites (NFC) (#95467)
Browse files Browse the repository at this point in the history
If NV == 0, nothing interesting happens after the "if" statement.  We
should just "continue" to the next value site.

While I am at it, this patch migrates a use of getValueForSite to
getValueArrayForSite.
  • Loading branch information
kazutakahirata authored Jun 13, 2024
1 parent 93181db commit 1365ce2
Showing 1 changed file with 15 additions and 15 deletions.
30 changes: 15 additions & 15 deletions llvm/tools/llvm-profdata/llvm-profdata.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2695,30 +2695,30 @@ static void traverseAllValueSites(const InstrProfRecord &Func, uint32_t VK,
uint32_t NS = Func.getNumValueSites(VK);
Stats.TotalNumValueSites += NS;
for (size_t I = 0; I < NS; ++I) {
uint32_t NV = Func.getNumValueDataForSite(VK, I);
std::unique_ptr<InstrProfValueData[]> VD = Func.getValueForSite(VK, I);
auto VD = Func.getValueArrayForSite(VK, I);
uint32_t NV = VD.size();
if (NV == 0)
continue;
Stats.TotalNumValues += NV;
if (NV) {
Stats.TotalNumValueSitesWithValueProfile++;
if (NV > Stats.ValueSitesHistogram.size())
Stats.ValueSitesHistogram.resize(NV, 0);
Stats.ValueSitesHistogram[NV - 1]++;
}
Stats.TotalNumValueSitesWithValueProfile++;
if (NV > Stats.ValueSitesHistogram.size())
Stats.ValueSitesHistogram.resize(NV, 0);
Stats.ValueSitesHistogram[NV - 1]++;

uint64_t SiteSum = 0;
for (uint32_t V = 0; V < NV; V++)
SiteSum += VD[V].Count;
for (const auto &V : VD)
SiteSum += V.Count;
if (SiteSum == 0)
SiteSum = 1;

for (uint32_t V = 0; V < NV; V++) {
for (const auto &V : VD) {
OS << "\t[ " << format("%2u", I) << ", ";
if (Symtab == nullptr)
OS << format("%4" PRIu64, VD[V].Value);
OS << format("%4" PRIu64, V.Value);
else
OS << Symtab->getFuncOrVarName(VD[V].Value);
OS << ", " << format("%10" PRId64, VD[V].Count) << " ] ("
<< format("%.2f%%", (VD[V].Count * 100.0 / SiteSum)) << ")\n";
OS << Symtab->getFuncOrVarName(V.Value);
OS << ", " << format("%10" PRId64, V.Count) << " ] ("
<< format("%.2f%%", (V.Count * 100.0 / SiteSum)) << ")\n";
}
}
}
Expand Down

0 comments on commit 1365ce2

Please sign in to comment.