[llvm-branch-commits] [llvm] [BOLT] Add pseudo probe inline tree to YAML profile (PR #107137)

Lei Wang via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Wed Sep 11 17:35:42 PDT 2024


================
@@ -95,24 +95,30 @@ template <> struct MappingTraits<bolt::SuccessorInfo> {
 
 namespace bolt {
 struct PseudoProbeInfo {
-  llvm::yaml::Hex64 GUID;
-  uint64_t Index;
-  uint8_t Type;
+  uint32_t InlineTreeIndex = 0;
+  uint64_t BlockMask = 0; // bitset with probe indices
----------------
wlei-llvm wrote:

How much can it save using the Mask? if the saving is not significant, just using `BlockProbes` might be good for the readability. 

> Assume BlockMask == 1 if no other probes are set

Could you clarify more on this? why we need a special use for the value `1` BlockMask, there still could be more probes? say we can have two probes: 1 basic block probe(ID1) and 1 call probe(ID2).



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


More information about the llvm-branch-commits mailing list