[llvm] [memprof] Add InstrProfWriter::addMemProfData (PR #116528)

Kazu Hirata via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 18 07:27:51 PST 2024


================
@@ -350,6 +350,31 @@ bool InstrProfWriter::addMemProfCallStack(
   return true;
 }
 
+bool InstrProfWriter::addMemProfData(memprof::IndexedMemProfData Incoming,
+                                     function_ref<void(Error)> Warn) {
+  if (MemProfData.Frames.empty())
----------------
kazutakahirata wrote:

Good idea!  Let me do that as soon as I remove `TEST_F(InstrProfTest, test_memprof_v0)`, which, being v0/v1, still adds two components.  I'm going to add a comment for now.

https://github.com/llvm/llvm-project/pull/116528


More information about the llvm-commits mailing list