diff --git a/llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp b/llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp index f82c29938b632..14ed40f16683a 100644 --- a/llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp +++ b/llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp @@ -678,7 +678,6 @@ Value *VPInstruction::generate(VPTransformState &State) { } // Reduce all of the unrolled parts into a single vector. Value *ReducedPartRdx = RdxParts[0]; - unsigned Op = RdxDesc.getOpcode(); if (PhiR->isOrdered()) { ReducedPartRdx = RdxParts[UF - 1]; } else { @@ -687,11 +686,12 @@ Value *VPInstruction::generate(VPTransformState &State) { Builder.setFastMathFlags(RdxDesc.getFastMathFlags()); for (unsigned Part = 1; Part < UF; ++Part) { Value *RdxPart = RdxParts[Part]; - if (Op != Instruction::ICmp && Op != Instruction::FCmp) - ReducedPartRdx = Builder.CreateBinOp( - (Instruction::BinaryOps)Op, RdxPart, ReducedPartRdx, "bin.rdx"); - else + if (RecurrenceDescriptor::isMinMaxRecurrenceKind(RK)) ReducedPartRdx = createMinMaxOp(Builder, RK, ReducedPartRdx, RdxPart); + else + ReducedPartRdx = + Builder.CreateBinOp((Instruction::BinaryOps)RdxDesc.getOpcode(), + RdxPart, ReducedPartRdx, "bin.rdx"); } }