Skip to content

Conversation

@teresajohnson
Copy link
Contributor

Remove unused collection of context size information that was likely
leftover from debugging / testing.

Remove unused collection of context size information that was likely
leftover from debugging / testing.
@llvmbot llvmbot added the llvm:analysis Includes value tracking, cost tables and constant folding label Dec 16, 2024
@llvmbot
Copy link
Member

llvmbot commented Dec 16, 2024

@llvm/pr-subscribers-llvm-analysis

Author: Teresa Johnson (teresajohnson)

Changes

Remove unused collection of context size information that was likely
leftover from debugging / testing.


Full diff: https://github.com/llvm/llvm-project/pull/120156.diff

1 Files Affected:

  • (modified) llvm/lib/Analysis/MemoryProfileInfo.cpp (-2)
diff --git a/llvm/lib/Analysis/MemoryProfileInfo.cpp b/llvm/lib/Analysis/MemoryProfileInfo.cpp
index 0a6623c67d6bb3..217f304e30de1a 100644
--- a/llvm/lib/Analysis/MemoryProfileInfo.cpp
+++ b/llvm/lib/Analysis/MemoryProfileInfo.cpp
@@ -164,8 +164,6 @@ void CallStackTrie::addCallStack(
   assert(Curr);
   Curr->ContextSizeInfo.insert(Curr->ContextSizeInfo.end(),
                                ContextSizeInfo.begin(), ContextSizeInfo.end());
-  std::vector<ContextTotalSize> AllContextSizeInfo;
-  collectContextSizeInfo(Curr, AllContextSizeInfo);
 }
 
 void CallStackTrie::addCallStack(MDNode *MIB) {

@teresajohnson teresajohnson merged commit bf700c3 into llvm:main Dec 17, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

llvm:analysis Includes value tracking, cost tables and constant folding

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants