[PATCH] D146999: [LV][NFC] Improve complexity of fixing users of recurrences

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 6 08:40:15 PDT 2023


fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.

LGTM, thanks!

I put up a patch that uses the VPLiveOuts directly, but that can act as follow-up: D147472 <https://reviews.llvm.org/D147472>



================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:3894
+  if (!Cost->requiresScalarEpilogue(VF)) {
+    llvm::SmallPtrSet<PHINode *, 2> ToFix;
+    for (User *U : Phi->users())
----------------
nit: no need to use `llvm::` (e.g. it is not used for other things in the namespace, like PHINode or User.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146999



More information about the llvm-commits mailing list