[llvm-branch-commits] [llvm] [LV] Bundle partial reductions inside VPExpressionRecipe (PR #147302)

[Nicholas Guy via llvm-branch-commits]

================
@@ -2955,12 +2966,14 @@ 
tryToMatchAndCreateMulAccumulateReduction(VPReductionRecipe *Red,
 
     // Match reduce.add(mul(ext, ext)).
     if (RecipeA && RecipeB &&
-        (RecipeA->getOpcode() == RecipeB->getOpcode() || A == B) &&
+        (RecipeA->getOpcode() == RecipeB->getOpcode() || A == B ||
----------------
NickGuy-Arm wrote:

Is there a case where the recipe opcodes could be different, but A and B are 
still equal? Do we need both checks here? The `|| A == B` feels redundant, but 
maybe I'm missing something.

https://github.com/llvm/llvm-project/pull/147302
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits