[llvm] Fix #86269: remove unused variable (PR #86927)

via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 28 03:16:04 PDT 2024


================
@@ -509,7 +509,7 @@ void TailRecursionEliminator::createTailRecurseLoopHeader(CallInst *CI) {
   BasicBlock *NewEntry = BasicBlock::Create(F.getContext(), "", &F, HeaderBB);
   NewEntry->takeName(HeaderBB);
   HeaderBB->setName("tailrecurse");
-  BranchInst *BI = BranchInst::Create(HeaderBB, NewEntry);
+  BranchInst::Create(HeaderBB, NewEntry);
   // If the new branch preserves the debug location of CI, it could result in
   // misleading stepping, if CI is located in a conditional branch.
   // So, here we don't give any debug location to BI.
----------------
mikaelholmen wrote:

Since there is no "BI" variable anymore it's confusing to reference it in the comment.
Perhaps change "BI" to "the new branch"?

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


More information about the llvm-commits mailing list