[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