Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Scalar] Remove an unused variable #105767

Merged
merged 1 commit into from
Aug 23, 2024

Conversation

kazutakahirata
Copy link
Contributor

The last use was removed by:

commit 89fe570
Author: Philip Reames listmail@philipreames.com
Date: Tue May 12 23:39:23 2015 +0000

The last use was removed by:

  commit 89fe570
  Author: Philip Reames <listmail@philipreames.com>
  Date:   Tue May 12 23:39:23 2015 +0000
@llvmbot
Copy link
Member

llvmbot commented Aug 23, 2024

@llvm/pr-subscribers-llvm-transforms

Author: Kazu Hirata (kazutakahirata)

Changes

The last use was removed by:

commit 89fe570
Author: Philip Reames <listmail@philipreames.com>
Date: Tue May 12 23:39:23 2015 +0000


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

1 Files Affected:

  • (modified) llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp (-1)
diff --git a/llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp b/llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp
index 77d67a2ce0f380..bf86be0dd387f0 100644
--- a/llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp
+++ b/llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp
@@ -342,7 +342,6 @@ bool PlaceSafepointsPass::runImpl(Function &F, const TargetLibraryInfo &TLI) {
         // The split loop structure here is so that we only need to recalculate
         // the dominator tree once.  Alternatively, we could just keep it up to
         // date and use a more natural merged loop.
-        SetVector<BasicBlock *> SplitBackedges;
         for (BasicBlock *Header : Headers) {
           BasicBlock *NewBB = SplitEdge(Term->getParent(), Header, &DT);
           PollsNeeded.push_back(NewBB->getTerminator());

@kazutakahirata kazutakahirata merged commit fdaaa87 into llvm:main Aug 23, 2024
8 of 10 checks passed
@kazutakahirata kazutakahirata deleted the cleanup_001_SplitBackedges branch August 23, 2024 04:27
cjdb pushed a commit to cjdb/llvm-project that referenced this pull request Aug 23, 2024
The last use was removed by:

  commit 89fe570
  Author: Philip Reames <listmail@philipreames.com>
  Date:   Tue May 12 23:39:23 2015 +0000
dmpolukhin pushed a commit to dmpolukhin/llvm-project that referenced this pull request Sep 2, 2024
The last use was removed by:

  commit 89fe570
  Author: Philip Reames <listmail@philipreames.com>
  Date:   Tue May 12 23:39:23 2015 +0000
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants