Temporarily revert "[SLP] fix miscompile on min/max reductions with extra uses (PR43948)"

as it causes an ICE on valid. A testcase was followed up on the original thread.

This reverts commit a3e61946c5bd7bdfab15af76b292e52d6ffa27f7.

1 files changed, 1 insertions, 14 deletions

diff --git a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
index 549d8501c78..1a7a59c47d6 100644
--- a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
+++ b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
@@ -6778,21 +6778,8 @@ public:
           VectorizedTree = VectReductionData.createOp(Builder, "op.extra", I);
         }
       }
-
-      // Update users. For a min/max reduction that ends with a compare and
-      // select, we also have to RAUW for the compare instruction feeding the
-      // reduction root. That's because the original compare may have extra uses
-      // besides the final select of the reduction.
-      if (ReductionData.isMinMax() && isa<SelectInst>(VectorizedTree)) {
-        assert(isa<SelectInst>(ReductionRoot) &&
-               "Expected min/max reduction to have select root instruction");
-
-        Value *ScalarCond = cast<SelectInst>(ReductionRoot)->getCondition();
-        Value *VectorCond = cast<SelectInst>(VectorizedTree)->getCondition();
-        ScalarCond->replaceAllUsesWith(VectorCond);
-      }
+      // Update users.
       ReductionRoot->replaceAllUsesWith(VectorizedTree);
-
       // Mark all scalar reduction ops for deletion, they are replaced by the
       // vector reductions.
       V.eraseInstructions(IgnoreList);