[llvm] [LV] NFCI: Add RecurKind to VPPartialReductionChain (PR #181705)

Sander de Smalen via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 19 02:54:07 PST 2026


================
@@ -6143,9 +6149,7 @@ void VPlanTransforms::createPartialReductions(VPlan &Plan,
     }
   }
 
-  for (auto &[Phi, Chains] : ChainsByPhi) {
-    RecurKind RK = cast<VPReductionPHIRecipe>(Phi)->getRecurrenceKind();
+  for (auto &[Phi, Chains] : ChainsByPhi)
----------------
sdesmalen-arm wrote:

There are probably different interpretations to the guidance to omit curly braces for simple single-statements, but I still consider the nested `for()` loop a single-line statement. I also find it easier to read without the curly braces.

https://github.com/llvm/llvm-project/pull/181705


More information about the llvm-commits mailing list