[llvm-branch-commits] [llvm] [BOLT][NFC] Simplify parseAggregatedLBREntry (PR #126998)

Amir Ayupov via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Wed Feb 12 21:18:14 PST 2025


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

>From a3ce5fb3b34ebddda7b68db1a8a288dcb729e319 Mon Sep 17 00:00:00 2001
From: Amir Ayupov <aaupov at fb.com>
Date: Wed, 12 Feb 2025 21:18:05 -0800
Subject: [PATCH] fix build

Created using spr 1.3.4
---
 bolt/lib/Profile/DataAggregator.cpp | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/bolt/lib/Profile/DataAggregator.cpp b/bolt/lib/Profile/DataAggregator.cpp
index f30f00ac1dd5e..2df2fa16ecf64 100644
--- a/bolt/lib/Profile/DataAggregator.cpp
+++ b/bolt/lib/Profile/DataAggregator.cpp
@@ -1255,11 +1255,10 @@ std::error_code DataAggregator::parseAggregatedLBREntry() {
 
   while (checkAndConsumeFS()) {
   }
-  ErrorOr<int64_t> FrequencyOrErr =
+  ErrorOr<int64_t> Frequency =
       parseNumberField(FieldSeparator, Type != AggregatedLBREntry::BRANCH);
-  if (std::error_code EC = FrequencyOrErr.getError())
+  if (std::error_code EC = Frequency.getError())
     return EC;
-  uint64_t Frequency = static_cast<uint64_t>(FrequencyOrErr.get());
 
   uint64_t Mispreds = 0;
   if (Type == AggregatedLBREntry::BRANCH) {
@@ -1283,7 +1282,9 @@ std::error_code DataAggregator::parseAggregatedLBREntry() {
     if (BF)
       BF->setHasProfileAvailable();
 
-  AggregatedLBRs.emplace_back(From.get(), To.get(), Frequency, Mispreds, Type);
+  AggregatedLBRs.emplace_back(AggregatedLBREntry{
+      From.get(), To.get(), static_cast<uint64_t>(Frequency.get()), Mispreds,
+      Type});
 
   return std::error_code();
 }



More information about the llvm-branch-commits mailing list