[llvm] [PseudoProbe] Extend to skip instrumenting probe into the dests of invoke (PR #79919)
via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 12 19:39:25 PST 2024
================
@@ -173,21 +173,98 @@ SampleProfileProber::SampleProfileProber(Function &Func,
BlockProbeIds.clear();
CallProbeIds.clear();
LastProbeId = (uint32_t)PseudoProbeReservedId::Last;
- computeProbeIdForBlocks();
- computeProbeIdForCallsites();
- computeCFGHash();
+
+ DenseSet<BasicBlock *> BlocksToIgnoreProbe;
+ DenseSet<BasicBlock *> BlocksToIgnoreCall;
----------------
WenleiHe wrote:
What's the main reason to split these into two containers? Is that because for "NewSplit" blocks, we want to ignore block probe but we still want to have call probe for calls from those blocks?
If that's the case, suggest following renaming:
BlocksToIgnoreCall -> BlocksAndCallsToIgnore
BlocksToIgnoreProbe -> BlocksToIgnore
https://github.com/llvm/llvm-project/pull/79919
More information about the llvm-commits
mailing list