[PATCH] D94435: [SampleFDO] Add the support to split the function profiles with context into separate sections.

[Hongtao Yu via Phabricator via llvm-commits]

hoy added inline comments.


================
Comment at: llvm/include/llvm/ProfileData/SampleProf.h:169
+  // Indicate the section contains only profile without context.
+  SecFlagNoContext = (1 << 1)
 };
----------------
Nit: how about name this `SecFlagNoCalleeContext` or `SecFlagFlat` or something? Just want to be a bit more clear as CS profile also uses the term context.


================
Comment at: llvm/include/llvm/ProfileData/SampleProfWriter.h:159
+const std::array<SmallVector<SecHdrTableEntry, 8>, NumOfLayout>
+    ExtBinaryLayoutTable = {
+        // Note that SecFuncOffsetTable section is written after SecLBRProfile
----------------
Nit: name it `ExtBinaryHdrLayoutTable`?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D94435