[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