[llvm] Bail out jump threading on indirect branches (PR #103688)

via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 15 13:05:00 PDT 2024


================
@@ -1029,11 +1031,12 @@ CanRedirectPredsOfEmptyBBToSucc(BasicBlock *BB, BasicBlock *Succ,
     return false;
 
   // Get single common predecessors of both BB and Succ
+  // TODO: Replace this with a proper `set intersect` algorithm
   for (BasicBlock *SuccPred : SuccPreds) {
     if (BBPreds.count(SuccPred)) {
+      CommonPred = SuccPred;
----------------
hiraditya wrote:

yeah, this is not needed. The second bug is somewhere else in simplifycfg, i'm still investigating.

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


More information about the llvm-commits mailing list