[PATCH] D139603: [llvm-profdata] Add option to cap profile output size

[NAKAMURA Takumi via Phabricator via llvm-commits]

chapuni added inline comments.


================
Comment at: llvm/lib/ProfileData/SampleProfWriter.cpp:125
+  OutputStream->write(StringBuffer.data(), StringBuffer.size());
+  LLVM_DEBUG(dbgs() << "Profile originally has " << OriginalFunctionCount
+                    << " functions, reduced to " << ProfileMap.size() << " in "
----------------
FYI, OriginalFunctionCount was unused but fixed in rG9f4a9d3f4450


================
Comment at: llvm/lib/ProfileData/SampleProfWriter.cpp:127
+                    << " functions, reduced to " << ProfileMap.size() << " in "
+                    << IterationCount << " iterations\n");
+  return sampleprof_error::success;
----------------
IterationCount is used only here.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139603/new/

https://reviews.llvm.org/D139603