[PATCH] D45366: Support generic expansion of ordered vector reduction (PR36732)

[Renato Golin via Phabricator via llvm-commits]

rengolin added a comment.

For now, that the shuffle reduction is in loop utils, this patch is fine. But this really ought to be in target transform info.



================
Comment at: lib/Transforms/Utils/LoopUtils.cpp:1531
+Value *
+llvm::getOrderedReduction(IRBuilder<> &Builder, Value *Acc, Value *Src,
+                          unsigned Op,
----------------
SVE will be able to do this with a single instruction, we should be able to override this based on target info.


Repository:
  rL LLVM

https://reviews.llvm.org/D45366