[PATCH] D90247: [AArch64] Add legalizations for VECREDUCE_SEQ_FADD

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

> Ok, I can build that out. Are we okay with the suboptimal legalization though? I'll wait for that decision before putting more time into this.
>
> Or does anyone see a clever fix for the illegal type legalization? It looks like we lost information during widening, so I'm not sure we can get it back in a non-hacky way.

Not sure I follow. If the neutral element is fixed, then the extra fadds should also get folded away. Or is there some additional sub-optimality here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90247