[PATCH] D114002: [LoopVectorize] Check the number of uses of an FAdd before classifying as ordered

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 18 07:35:43 PST 2021


dmgreen added a comment.

Yeah that sounds fine. I took another look and checkOrderedReduction is already checking for a small subset of what getReductionOpChain is testing for. It would be nice that if we fixed getReductionOpChain to not require a single user it would just work without extra changes, but it's simple enough to fix both places.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114002



More information about the llvm-commits mailing list