[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