[PATCH] D117307: [LoopVectorize] Only check for ordered reductions if the op chain was found

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 20 01:59:45 PST 2022


david-arm accepted this revision.
david-arm added a comment.
This revision is now accepted and ready to land.

LGTM! Thanks for addressing my comments @kmclaughlin. :)



================
Comment at: llvm/lib/Analysis/IVDescriptors.cpp:1041
 SmallVector<Instruction *, 4>
-RecurrenceDescriptor::getReductionOpChain(PHINode *Phi, Loop *L) const {
+RecurrenceDescriptor::findReductionOpChain(PHINode *Phi, Loop *L,
+                                           Instruction *LoopExitInstr,
----------------
nit: Could you fix the formatting before merging? Thanks!


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

https://reviews.llvm.org/D117307



More information about the llvm-commits mailing list