[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