[PATCH] D155845: [VPlan] Fix in-loop reduction chains using VPlan def-use chains (NFCI)

Ayal Zaks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 22 23:07:46 PDT 2023


Ayal added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:9156
+    Phis.push_back(&R);
+
+  for (VPRecipeBase *R : Phis) {
----------------
Ayal wrote:
> Bypass this `for` loop if MinVF.isScalar()?
> Bypass this for loop if MinVF.isScalar()?
Ah, isOrdered reductions are considered inLoop even for VF=1 (but only them).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D155845/new/

https://reviews.llvm.org/D155845



More information about the llvm-commits mailing list