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

[William Junda Huang via Phabricator via llvm-commits]

huangjd marked an inline comment as done.
huangjd added inline comments.


================
Comment at: llvm/include/llvm/ProfileData/SampleProfWriter.h:74
+  /// from llvm-profdata command line arguments. Ignore transient states (those
+  /// always being set by write() before use).
+  virtual void reset(std::unique_ptr<raw_ostream> &OS) {
----------------
snehasish wrote:
> It looks like derived classes must always call this function. Can you add a comment here for the future? Maybe something like `// This function must always be called by the overridden implementation`? 
It is not called by derived classes. It is called by llvm-profdata if the writer needs to be reused 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139603