================ @@ -2744,6 +2702,12 @@ class VPSingleDefBundleRecipe : public VPSingleDefRecipe { /// vector operands, performing a reduction.add on the result, and adding /// the scalar result to a chain. MulAccumulateReduction, + /// Represent an inloop multiply-accumulate reduction, multiplying the + /// extended vector operands, negating the multiplication, performing a + /// reduction.add + /// on the result, and adding + /// the scalar result to a chain. + ExtNegatedMulAccumulateReduction, ---------------- fhahn wrote:
Given that `VPPartialReductionRecipe` is already a sub-class of `VPReductionRecipe`, I think it should be possible to first add support for partial reductions to bundle/expression recipes, without also merging `VPPartialReductionRecipe`. This would simplify the patches and clearly separate the functional from the non-functional changes/refactorings. It looks like most of the code for supporting partial reductions already needs to explicitly check for partial reductions, so hopefully such a split wouldn't complicate thinga https://github.com/llvm/llvm-project/pull/146073 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits