[PATCH] D119661: [LV] Support chained phis as incoming values for first-order recurs.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 1 03:53:18 PDT 2022


fhahn marked an inline comment as done.
fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:8869
+      // values.
+      recordRecipeOf(Phi);
+
----------------
TKaipeng wrote:
> Should we record recipes for induction phis? It's possible to be the incoming value for later phis.
Yes, that's a good point, thanks! Should be fixed in the latest version and I added some extra test.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119661



More information about the llvm-commits mailing list