[llvm] Fix #86269: remove unused variable (PR #86927)
Shan Huang via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 28 03:17:26 PDT 2024
https://github.com/Apochens updated https://github.com/llvm/llvm-project/pull/86927
>From d172bf68387a90c7f353d5afa40c494a3b218717 Mon Sep 17 00:00:00 2001
From: Apochens <52285902006 at stu.ecnu.edu.cn>
Date: Thu, 28 Mar 2024 10:08:42 +0000
Subject: [PATCH 1/2] remove unused variable
---
llvm/lib/Transforms/Scalar/TailRecursionElimination.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/llvm/lib/Transforms/Scalar/TailRecursionElimination.cpp b/llvm/lib/Transforms/Scalar/TailRecursionElimination.cpp
index 34d39f3fe6dcac..10c92395b4777a 100644
--- a/llvm/lib/Transforms/Scalar/TailRecursionElimination.cpp
+++ b/llvm/lib/Transforms/Scalar/TailRecursionElimination.cpp
@@ -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.
>From 1dd309d82aa50fb71b52b2f60dba15918e046592 Mon Sep 17 00:00:00 2001
From: Apochens <52285902006 at stu.ecnu.edu.cn>
Date: Thu, 28 Mar 2024 10:17:14 +0000
Subject: [PATCH 2/2] refine the description
---
llvm/lib/Transforms/Scalar/TailRecursionElimination.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/llvm/lib/Transforms/Scalar/TailRecursionElimination.cpp b/llvm/lib/Transforms/Scalar/TailRecursionElimination.cpp
index 10c92395b4777a..bc4b6de2f07f1f 100644
--- a/llvm/lib/Transforms/Scalar/TailRecursionElimination.cpp
+++ b/llvm/lib/Transforms/Scalar/TailRecursionElimination.cpp
@@ -512,7 +512,7 @@ void TailRecursionEliminator::createTailRecurseLoopHeader(CallInst *CI) {
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.
+ // So, here we don't give any debug location to the new branch.
// Move all fixed sized allocas from HeaderBB to NewEntry.
for (BasicBlock::iterator OEBI = HeaderBB->begin(), E = HeaderBB->end(),
More information about the llvm-commits
mailing list