[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