[Attributor][FIX] Register right new created BB.

[lcvon007]

CBBB will keep same after the first iteration so registerManifestAddedBasicBlock would always register the same basic block later.

[llvmbot]

@llvm/pr-subscribers-llvm-transforms

Author: None (lcvon007)

Changes

---

Full diff: https://github.com/llvm/llvm-project/pull/84929.diff

1 Files Affected:

• (modified) llvm/lib/Transforms/IPO/AttributorAttributes.cpp (+1-1)

diff --git a/llvm/lib/Transforms/IPO/AttributorAttributes.cpp b/llvm/lib/Transforms/IPO/AttributorAttributes.cpp
index 488a6f0bb153a4..f98833bd119891 100644
--- a/llvm/lib/Transforms/IPO/AttributorAttributes.cpp
+++ b/llvm/lib/Transforms/IPO/AttributorAttributes.cpp
@@ -12371,7 +12371,7 @@ struct AAIndirectCallInfoCallSite : public AAIndirectCallInfo {
           SplitBlockAndInsertIfThen(LastCmp, IP, /* Unreachable */ false);
       BasicBlock *CBBB = CB->getParent();
       A.registerManifestAddedBasicBlock(*ThenTI->getParent());
-      A.registerManifestAddedBasicBlock(*CBBB);
+      A.registerManifestAddedBasicBlock(*IP->getParent());
       auto *SplitTI = cast<BranchInst>(LastCmp->getNextNode());
       BasicBlock *ElseBB;
       if (&*IP == CB) {

[shiltian]

LG. No tests need to be updated?

[lcvon007]

LG. No tests need to be updated?

yes, it doesn't affect the testcase. Your approval is removed by mistake by me, and please help give an approval again if it's easy to do it, thank you very much. @shiltian