aboutsummaryrefslogtreecommitdiff
path: root/llvm/unittests/ProfileData/MemProfTest.cpp
diff options
context:
space:
mode:
authorKazu Hirata <kazu@google.com>2024-12-07 10:41:55 -0800
committerGitHub <noreply@github.com>2024-12-07 10:41:55 -0800
commit8eb5baf5ea52edd0b8df38ffca2a65e21610f3ad (patch)
tree5bb335f4b83ea4571b16e6e007b5241e79200f8b /llvm/unittests/ProfileData/MemProfTest.cpp
parent4cf0bd89eed3ca08fc00c38a0419ae514075ee7c (diff)
downloadllvm-8eb5baf5ea52edd0b8df38ffca2a65e21610f3ad.zip
llvm-8eb5baf5ea52edd0b8df38ffca2a65e21610f3ad.tar.gz
llvm-8eb5baf5ea52edd0b8df38ffca2a65e21610f3ad.tar.bz2
[memprof] Use IndexedMemProfData in a unit test (NFC) (#119062)
IndexedMemProfData eliminates the need for the "using" directives. Also, we do not need to declare maps for individual components of the MemProf profile.
Diffstat (limited to 'llvm/unittests/ProfileData/MemProfTest.cpp')
-rw-r--r--llvm/unittests/ProfileData/MemProfTest.cpp32
1 files changed, 12 insertions, 20 deletions
diff --git a/llvm/unittests/ProfileData/MemProfTest.cpp b/llvm/unittests/ProfileData/MemProfTest.cpp
index 04e9141..5e24579 100644
--- a/llvm/unittests/ProfileData/MemProfTest.cpp
+++ b/llvm/unittests/ProfileData/MemProfTest.cpp
@@ -549,12 +549,6 @@ TEST(MemProf, IndexedMemProfRecordToMemProfRecord) {
EXPECT_EQ(Record.CallSites[1][1].hash(), F4.hash());
}
-using FrameIdMapTy =
- llvm::DenseMap<::llvm::memprof::FrameId, ::llvm::memprof::Frame>;
-using CallStackIdMapTy =
- llvm::DenseMap<::llvm::memprof::CallStackId,
- ::llvm::SmallVector<::llvm::memprof::FrameId>>;
-
// Populate those fields returned by getHotColdSchema.
MemInfoBlock makePartialMIB() {
MemInfoBlock MIB;
@@ -575,12 +569,11 @@ TEST(MemProf, MissingCallStackId) {
IndexedMR.AllocSites.push_back(AI);
// Create empty maps.
- const FrameIdMapTy IdToFrameMap;
- const CallStackIdMapTy CSIdToCallStackMap;
- llvm::memprof::FrameIdConverter<decltype(IdToFrameMap)> FrameIdConv(
- IdToFrameMap);
- llvm::memprof::CallStackIdConverter<decltype(CSIdToCallStackMap)> CSIdConv(
- CSIdToCallStackMap, FrameIdConv);
+ IndexedMemProfData MemProfData;
+ llvm::memprof::FrameIdConverter<decltype(MemProfData.Frames)> FrameIdConv(
+ MemProfData.Frames);
+ llvm::memprof::CallStackIdConverter<decltype(MemProfData.CallStacks)>
+ CSIdConv(MemProfData.CallStacks, FrameIdConv);
// We are only interested in errors, not the return value.
(void)IndexedMR.toMemProfRecord(CSIdConv);
@@ -597,15 +590,14 @@ TEST(MemProf, MissingFrameId) {
IndexedMemProfRecord IndexedMR;
IndexedMR.AllocSites.push_back(AI);
- // An empty map to trigger a mapping error.
- const FrameIdMapTy IdToFrameMap;
- CallStackIdMapTy CSIdToCallStackMap;
- CSIdToCallStackMap.insert({0x222, {2, 3}});
+ // An empty Frame map to trigger a mapping error.
+ IndexedMemProfData MemProfData;
+ MemProfData.CallStacks.insert({0x222, {2, 3}});
- llvm::memprof::FrameIdConverter<decltype(IdToFrameMap)> FrameIdConv(
- IdToFrameMap);
- llvm::memprof::CallStackIdConverter<decltype(CSIdToCallStackMap)> CSIdConv(
- CSIdToCallStackMap, FrameIdConv);
+ llvm::memprof::FrameIdConverter<decltype(MemProfData.Frames)> FrameIdConv(
+ MemProfData.Frames);
+ llvm::memprof::CallStackIdConverter<decltype(MemProfData.CallStacks)>
+ CSIdConv(MemProfData.CallStacks, FrameIdConv);
// We are only interested in errors, not the return value.
(void)IndexedMR.toMemProfRecord(CSIdConv);