[llvm] [InstCombine] Optimistically allow multiple shufflevector uses in foldOpPhi (PR #114278)

Yingwei Zheng via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 6 23:41:15 PST 2024


================
@@ -1773,17 +1773,33 @@ Instruction *InstCombinerImpl::foldOpIntoPhi(Instruction &I, PHINode *PN) {
   if (NumPHIValues == 0)
     return nullptr;
 
-  // We normally only transform phis with a single use.  However, if a PHI has
-  // multiple uses and they are all the same operation, we can fold *all* of the
-  // uses into the PHI.
+  // We normally only transform phis with a single use.
+  bool AllUsesIdentical = false;
+  bool MultipleUses = false;
----------------
dtcxzyw wrote:

`MultipleUses` is misleading. `MultipleShuffleVectorUses` would be better.


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


More information about the llvm-commits mailing list