[PATCH] D89162: [SVE] Lower fixed length VECREDUCE_SEQ_FADD operation

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm added a comment.

@cameron.mcinally: I'm sure you know this but just in case it saves some time I can confirm you will not hit the splitting code until after you relax shouldExpandReduction.  So I think your current patch is complete so it really comes down to whether adding the support this way round (i.e. only allow legal types for SVE, then allow all types and implement the legalisation) is acceptable to @nikic .


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

https://reviews.llvm.org/D89162