[llvm] [BOLT][NFC] Simplify BBHashMapTy (PR #91812)
Amir Ayupov via llvm-commits
llvm-commits at lists.llvm.org
Wed May 22 14:26:00 PDT 2024
https://github.com/aaupov updated https://github.com/llvm/llvm-project/pull/91812
>From 78ef87fcf63f3695560399b036b8e265c3e39396 Mon Sep 17 00:00:00 2001
From: Amir Ayupov <aaupov at fb.com>
Date: Fri, 10 May 2024 14:16:57 -0700
Subject: [PATCH] [BOLT][NFC] Simplify BBHashMapTy
Make EntryTy a thin wrapper struct.
---
bolt/include/bolt/Profile/BoltAddressTranslation.h | 13 ++++---------
bolt/lib/Profile/DataAggregator.cpp | 4 ++--
2 files changed, 6 insertions(+), 11 deletions(-)
diff --git a/bolt/include/bolt/Profile/BoltAddressTranslation.h b/bolt/include/bolt/Profile/BoltAddressTranslation.h
index dcbcd0f60114f..3e68312b92c3a 100644
--- a/bolt/include/bolt/Profile/BoltAddressTranslation.h
+++ b/bolt/include/bolt/Profile/BoltAddressTranslation.h
@@ -183,14 +183,9 @@ class BoltAddressTranslation {
public:
/// Map basic block input offset to a basic block index and hash pair.
class BBHashMapTy {
- class EntryTy {
+ struct EntryTy {
unsigned Index;
size_t Hash;
-
- public:
- unsigned getBBIndex() const { return Index; }
- size_t getBBHash() const { return Hash; }
- EntryTy(unsigned Index, size_t Hash) : Index(Index), Hash(Hash) {}
};
std::map<uint32_t, EntryTy> Map;
@@ -206,15 +201,15 @@ class BoltAddressTranslation {
}
unsigned getBBIndex(uint32_t BBInputOffset) const {
- return getEntry(BBInputOffset).getBBIndex();
+ return getEntry(BBInputOffset).Index;
}
size_t getBBHash(uint32_t BBInputOffset) const {
- return getEntry(BBInputOffset).getBBHash();
+ return getEntry(BBInputOffset).Hash;
}
void addEntry(uint32_t BBInputOffset, unsigned BBIndex, size_t BBHash) {
- Map.emplace(BBInputOffset, EntryTy(BBIndex, BBHash));
+ Map.emplace(BBInputOffset, EntryTy{BBIndex, BBHash});
}
size_t getNumBasicBlocks() const { return Map.size(); }
diff --git a/bolt/lib/Profile/DataAggregator.cpp b/bolt/lib/Profile/DataAggregator.cpp
index c0fd69b98c82d..9d7d793ccb8c2 100644
--- a/bolt/lib/Profile/DataAggregator.cpp
+++ b/bolt/lib/Profile/DataAggregator.cpp
@@ -2355,7 +2355,7 @@ std::error_code DataAggregator::writeBATYAML(BinaryContext &BC,
YamlBB.Index = Idx;
for (auto BI = BlockMap.begin(), BE = BlockMap.end(); BI != BE; ++BI)
- YamlBF.Blocks[BI->second.getBBIndex()].Hash = BI->second.getBBHash();
+ YamlBF.Blocks[BI->second.Index].Hash = BI->second.Hash;
// Lookup containing basic block offset and index
auto getBlock = [&BlockMap](uint32_t Offset) {
@@ -2365,7 +2365,7 @@ std::error_code DataAggregator::writeBATYAML(BinaryContext &BC,
exit(1);
}
--BlockIt;
- return std::pair(BlockIt->first, BlockIt->second.getBBIndex());
+ return std::pair(BlockIt->first, BlockIt->second.Index);
};
for (const BranchInfo &BI : Branches.Data) {
More information about the llvm-commits
mailing list