[llvm] [memprof] Add MemProf version (PR #86414)

Snehasish Kumar via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 26 14:59:34 PDT 2024


================
@@ -75,11 +75,16 @@ class InstrProfWriter {
   // deployment of newer versions of llvm-profdata.
   bool WritePrevVersion = false;
 
+  // The version we should use to write MemProf in.
+  memprof::MemProfVersion MemProfVersionRequested;
+
 public:
   InstrProfWriter(bool Sparse = false,
                   uint64_t TemporalProfTraceReservoirSize = 0,
                   uint64_t MaxTemporalProfTraceLength = 0,
-                  bool WritePrevVersion = false);
+                  bool WritePrevVersion = false,
----------------
snehasish wrote:

Do we need WritePrevVersion? I don't see it being used in the patch. External APIs can directly specify the version once this patch lands so I think WritePrevVersion may be dropped?

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


More information about the llvm-commits mailing list