Skip to content

Commit 3061fed

Browse files
authored
[BOLT][NFC] Simplify FuncHashesTy (#91815)
Make EntryTy a thin wrapper struct.
1 parent a79acb0 commit 3061fed

File tree

1 file changed

+5
-10
lines changed

1 file changed

+5
-10
lines changed

bolt/include/bolt/Profile/BoltAddressTranslation.h

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -227,14 +227,9 @@ class BoltAddressTranslation {
227227

228228
/// Map function output address to its hash and basic blocks hash map.
229229
class FuncHashesTy {
230-
class EntryTy {
230+
struct EntryTy {
231231
size_t Hash;
232232
BBHashMapTy BBHashMap;
233-
234-
public:
235-
size_t getBFHash() const { return Hash; }
236-
const BBHashMapTy &getBBHashMap() const { return BBHashMap; }
237-
EntryTy(size_t Hash) : Hash(Hash) {}
238233
};
239234

240235
std::unordered_map<uint64_t, EntryTy> Map;
@@ -246,23 +241,23 @@ class BoltAddressTranslation {
246241

247242
public:
248243
size_t getBFHash(uint64_t FuncOutputAddress) const {
249-
return getEntry(FuncOutputAddress).getBFHash();
244+
return getEntry(FuncOutputAddress).Hash;
250245
}
251246

252247
const BBHashMapTy &getBBHashMap(uint64_t FuncOutputAddress) const {
253-
return getEntry(FuncOutputAddress).getBBHashMap();
248+
return getEntry(FuncOutputAddress).BBHashMap;
254249
}
255250

256251
void addEntry(uint64_t FuncOutputAddress, size_t BFHash) {
257-
Map.emplace(FuncOutputAddress, EntryTy(BFHash));
252+
Map.emplace(FuncOutputAddress, EntryTy{BFHash, BBHashMapTy()});
258253
}
259254

260255
size_t getNumFunctions() const { return Map.size(); };
261256

262257
size_t getNumBasicBlocks() const {
263258
size_t NumBasicBlocks{0};
264259
for (auto &I : Map)
265-
NumBasicBlocks += I.second.getBBHashMap().getNumBasicBlocks();
260+
NumBasicBlocks += I.second.BBHashMap.getNumBasicBlocks();
266261
return NumBasicBlocks;
267262
}
268263
};

0 commit comments

Comments
 (0)