[llvm] [PseudoProbe] Extend to skip instrumenting probe into the dests of invoke (PR #79919)

Lei Wang via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 30 13:58:26 PST 2024


================
@@ -16,6 +16,7 @@ namespace llvm {
 /// Compute a list of blocks that are only reachable via EH paths.
 template <typename FunctionT, typename BlockT>
 static void computeEHOnlyBlocks(FunctionT &F, DenseSet<BlockT *> &EHBlocks) {
+  assert(EHBlocks.empty() && "Output set should be empty");
----------------
wlei-llvm wrote:

Good point, then going to remove the `EHBlocks.clear()`, I think it should be user's responsibility to choose the right input, the function should avoid this side effect. 

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


More information about the llvm-commits mailing list