================ @@ -2512,9 +2507,11 @@ class VPMulAccumulateReductionRecipe : public VPReductionRecipe { MulAcc->getCondOp(), MulAcc->isOrdered(), WrapFlagsTy(MulAcc->hasNoUnsignedWrap(), MulAcc->hasNoSignedWrap()), MulAcc->getDebugLoc()), - ExtOp(MulAcc->getExtOpcode()), IsNonNeg(MulAcc->isNonNeg()), ResultTy(MulAcc->getResultType()), - IsPartialReduction(MulAcc->isPartialReduction()) {} + IsPartialReduction(MulAcc->isPartialReduction()) { + VecOpInfo[0] = MulAcc->getVecOp0Info(); + VecOpInfo[1] = MulAcc->getVecOp1Info(); + } ---------------- gbossu wrote:
Probably a stupid question because I'm not familiar with `VPlan`, but is there a reason why this isn't a more standard copy constructor, i.e. taking a `const VPMulAccumulateReductionRecipe &` as parameter? https://github.com/llvm/llvm-project/pull/136997 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits