[llvm] [BOLT] Drop parsing sample PC when processing LBR perf data (PR #123420)
Alexander Yermolovich via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 20 17:05:05 PST 2025
================
@@ -1472,9 +1383,9 @@ std::error_code DataAggregator::printLBRHeatMap() {
uint64_t DataAggregator::parseLBRSample(const PerfBranchSample &Sample,
bool NeedsSkylakeFix) {
uint64_t NumTraces{0};
- // LBRs are stored in reverse execution order. NextPC refers to the next
- // recorded executed PC.
- uint64_t NextPC = opts::UseEventPC ? Sample.PC : 0;
+ // LBRs are stored in reverse execution order. NextLBR refers to the next
+ // executed branch record.
+ const LBREntry *NextLBR{nullptr};
----------------
ayermolo wrote:
= nullptr for consistency?
https://github.com/llvm/llvm-project/pull/123420
More information about the llvm-commits
mailing list