[llvm-branch-commits] [llvm] [BOLT] Check duplicate top-of-brstack entry erratum (PR #147971)

[Amir Ayupov via llvm-branch-commits]

https://github.com/aaupov updated https://github.com/llvm/llvm-project/pull/147971

>From 390ad2fe3e0b0fd72ac37468c1e3482ff017cec2 Mon Sep 17 00:00:00 2001
From: Amir Ayupov <aaupov at fb.com>
Date: Thu, 10 Jul 2025 08:17:11 -0700
Subject: [PATCH] format

Created using spr 1.3.4
---
 bolt/lib/Profile/DataAggregator.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/bolt/lib/Profile/DataAggregator.cpp b/bolt/lib/Profile/DataAggregator.cpp
index 8407a8d95a856..07dda53c02949 100644
--- a/bolt/lib/Profile/DataAggregator.cpp
+++ b/bolt/lib/Profile/DataAggregator.cpp
@@ -566,7 +566,7 @@ void DataAggregator::checkLBRTOSErratum() {
       break;
     }
     // Disassemble instructions in [From, To] (no CFG is available yet)
-    for (uint64_t Addr = T.From; Addr < T.To; ) {
+    for (uint64_t Addr = T.From; Addr < T.To;) {
       // Since we checked the range first, there should be no unconditional jump
       // in the map. We can use entries as size hints to avoid disassembling.
       auto UII = UJmpOrSize.find(Addr);
@@ -660,7 +660,6 @@ Error DataAggregator::preprocessProfile(BinaryContext &BC) {
   if (opts::ImputeTraceFallthrough)
     imputeFallThroughs();
 
-  clear(Returns);
   clear(UncondJumps);
 
   if (opts::HeatmapMode) {