================ @@ -478,10 +675,34 @@ matchWeightsByHashes(BinaryContext &BC, Blocks.push_back(&Func.Blocks[I + 1]); BlendedBlockHash BlendedHash(BB->getHash()); BlendedHashes.push_back(BlendedHash); + // Collects pseudo probes attached to the BB for use in the StaleMatcher. + if (opts::ProfileUsePseudoProbes && + opts::StaleMatchingWithBlockPseudoProbes && PseudoProbeDecoder) { + const AddressProbesMap &ProbeMap = + PseudoProbeDecoder->getAddress2ProbesMap(); + const uint64_t FuncAddr = BF.getAddress(); + const std::pair<uint64_t, uint64_t> &BlockRange = + BB->getInputAddressRange(); + const auto &BlockProbes = + llvm::make_range(ProbeMap.lower_bound(FuncAddr + BlockRange.first), + ProbeMap.lower_bound(FuncAddr + BlockRange.second)); + for (const auto &[_, Probes] : BlockProbes) { + for (const MCDecodedPseudoProbe &Probe : Probes) { + if (Probe.getType() != static_cast<uint8_t>(PseudoProbeType::Block)) + continue; + if (Probe.getInlineTreeNode()->hasInlineSite()) + Matcher.mapGUIDAndIndexToProbe(Probe.getGuid(), Probe.getIndex(), + &Probe); + else + Matcher.mapIndexToProbe(Probe.getIndex(), &Probe); ---------------- wlei-llvm wrote:
Wondering why we need to use two containers? iiuc, all the probes contain the `GUID`, the top-level probes should contain the function GUID. https://github.com/llvm/llvm-project/pull/99891 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits