[llvm] [memprof] Reduce schema for Version2 (PR #89876)

Snehasish Kumar via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 24 09:26:31 PDT 2024


================
@@ -117,14 +117,21 @@ struct PortableMemInfoBlock {
   void clear() { *this = PortableMemInfoBlock(); }
 
   // Returns the full schema currently in use.
-  static MemProfSchema getSchema() {
+  static MemProfSchema getFullSchema() {
     MemProfSchema List;
 #define MIBEntryDef(NameTag, Name, Type) List.push_back(Meta::Name);
 #include "llvm/ProfileData/MIBEntryDef.inc"
 #undef MIBEntryDef
     return List;
   }
 
+  // Returns the schema consisting of the fields currently consumed by the
+  // compiler.
+  static MemProfSchema getSchema() {
----------------
snehasish wrote:

Maybe we should qualify the schema based on the use case. For example these fields are necessary for hot-cold annotation, so name this `getHotColdSchema()`?

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


More information about the llvm-commits mailing list