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

[Cameron McInally via Phabricator via llvm-commits]

cameron.mcinally added a comment.

In D89162#2350619 <https://reviews.llvm.org/D89162#2350619>, @nikic wrote:

> The eventual goal here is to expand reductions during legalization only. The IR expansion exists because the DAG legalization support has been patchy historically, with VECREDUCE_SEQ_* being the last remaining hole.

Ah, ok. I'm happy to hear we're all in agreement. I might be able to volunteer some time for that work, but it's not really my prime directive.

Again, apologies for the mix up. There were a lot of moving parts for this patch and I dropped the ball. *^_^*


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

https://reviews.llvm.org/D89162