[llvm] [memprof] Remove a stale comment in a unit test (PR #119060)
Kazu Hirata via llvm-commits
llvm-commits at lists.llvm.org
Sat Dec 7 00:24:02 PST 2024
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/119060
We've removed IndexedAllocationInfo::CallStack, so we don't need to
mention it.
>From c56ce7a195519736cb3eda5e20d8ec205f5d4eeb Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Fri, 6 Dec 2024 23:45:21 -0800
Subject: [PATCH] [memprof] Remove a stale comment in a unit test
We've removed IndexedAllocationInfo::CallStack, so we don't need to
mention it.
---
llvm/unittests/ProfileData/InstrProfTest.cpp | 2 --
1 file changed, 2 deletions(-)
diff --git a/llvm/unittests/ProfileData/InstrProfTest.cpp b/llvm/unittests/ProfileData/InstrProfTest.cpp
index 273b89ca26aa98..49a186cc98df28 100644
--- a/llvm/unittests/ProfileData/InstrProfTest.cpp
+++ b/llvm/unittests/ProfileData/InstrProfTest.cpp
@@ -401,8 +401,6 @@ makeRecordV2(std::initializer_list<::llvm::memprof::CallStackId> AllocFrames,
const MemInfoBlock &Block, const memprof::MemProfSchema &Schema) {
llvm::memprof::IndexedMemProfRecord MR;
for (const auto &CSId : AllocFrames)
- // We don't populate IndexedAllocationInfo::CallStack because we use it only
- // in Version1.
MR.AllocSites.emplace_back(CSId, Block, Schema);
for (const auto &CSId : CallSiteFrames)
MR.CallSiteIds.push_back(CSId);
More information about the llvm-commits
mailing list