[PATCH] D93556: [CSSPGO][llvm-profgen] Compress recursive cycles in calling context

[Hongtao Yu via Phabricator via llvm-commits]

hoy added inline comments.


================
Comment at: llvm/tools/llvm-profgen/ProfileGenerator.h:63
+    compressRecursionContext<T>(
+        Context, CompressionSize == -1 ? Context.size() : CompressionSize);
+  }
----------------
Moving the implementation to ProfileGenerator.cpp so that `RecursionCompression` can be checked instead of using an extra global?


================
Comment at: llvm/tools/llvm-profgen/ProfileGenerator.h:65
+  }
+  // Remove adjacent repeated context sequences up to a given sequene length.
+  // Note that repeated sequences are identified based on the exact call site,
----------------
typo: sequence


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93556