[llvm] [BasicBlockSections] Introduce the path cloning profile format to BasicBlockSectionsProfileReader. (PR #67214)

Sriraman Tallam via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 11 15:02:35 PDT 2023


================
@@ -216,17 +191,17 @@ assignSections(MachineFunction &MF,
     // With the 'all' option, every basic block is placed in a unique section.
     // With the 'list' option, every basic block is placed in a section
     // associated with its cluster, unless we want individual unique sections
-    // for every basic block in this function (if FuncBBClusterInfo is empty).
+    // for every basic block in this function (if BBProfilesByBBID is empty).
     if (MF.getTarget().getBBSectionsType() == llvm::BasicBlockSection::All ||
-        FuncBBClusterInfo.empty()) {
+        BBProfilesByBBID.empty()) {
----------------
tmsri wrote:

Same deal here, FuncBBClusterInfo is a better name than BBProfilesByBBID.  I would like this name to capture the fact that it is regarding block layout.

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


More information about the llvm-commits mailing list