[clang] [llvm] [ubsan] Parse and use in LowerAllowCheckPass (PR #124211)

[Thurston Dang via cfe-commits]

================
@@ -107,7 +108,18 @@ static bool removeUbsanTraps(Function &F, const 
BlockFrequencyInfo &BFI,
       case Intrinsic::allow_runtime_check: {
         ++NumChecksTotal;
 
-        bool ToRemove = ShouldRemove(BB);
+        bool ToRemove = ShouldRemoveRandom();
+
+        unsigned int cutoff = 0;
----------------
thurstond wrote:

Done. I ended up using both BB and II:
```
  auto ShouldRemove = [&](const BasicBlock &BB, const IntrinsicInst *II) {
    unsigned int cutoff = GetCutoff(II);
    return ShouldRemoveRandom() || ShouldRemoveHot(BB, cutoff);
  };
```

https://github.com/llvm/llvm-project/pull/124211
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits