Skip to content

[Unifomity] Remove unused PhiInput definition (NFC)#161116

Merged
perlfu merged 1 commit into
llvm:mainfrom
perlfu:uniformity-remove-phi-input
Sep 30, 2025
Merged

[Unifomity] Remove unused PhiInput definition (NFC)#161116
perlfu merged 1 commit into
llvm:mainfrom
perlfu:uniformity-remove-phi-input

Conversation

@perlfu

@perlfu perlfu commented Sep 29, 2025

Copy link
Copy Markdown
Contributor

This appears to have no users.
Putting this to review in case there is some use I am unaware of.

@llvmbot

llvmbot commented Sep 29, 2025

Copy link
Copy Markdown
Member

@llvm/pr-subscribers-llvm-adt

Author: Carl Ritson (perlfu)

Changes

This appears to have no users.
Putting this to review in case there is some use I am unaware of.


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

1 Files Affected:

  • (modified) llvm/include/llvm/ADT/GenericUniformityImpl.h (-9)
diff --git a/llvm/include/llvm/ADT/GenericUniformityImpl.h b/llvm/include/llvm/ADT/GenericUniformityImpl.h
index 141816c304397..7fb0dbe22f12f 100644
--- a/llvm/include/llvm/ADT/GenericUniformityImpl.h
+++ b/llvm/include/llvm/ADT/GenericUniformityImpl.h
@@ -408,15 +408,6 @@ template <typename ContextT> class GenericUniformityAnalysisImpl {
                                 const CycleT *);
 
 protected:
-  /// \brief Value/block pair representing a single phi input.
-  struct PhiInput {
-    ConstValueRefT value;
-    BlockT *predBlock;
-
-    PhiInput(ConstValueRefT value, BlockT *predBlock)
-        : value(value), predBlock(predBlock) {}
-  };
-
   const ContextT &Context;
   const FunctionT &F;
   const CycleInfoT &CI;

@perlfu perlfu merged commit 55054db into llvm:main Sep 30, 2025
11 checks passed
@perlfu perlfu deleted the uniformity-remove-phi-input branch September 30, 2025 08:42
mahesh-attarde pushed a commit to mahesh-attarde/llvm-project that referenced this pull request Oct 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants