[PATCH] D90247: [AArch64] Add legalizations for VECREDUCE_SEQ_FADD
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 27 10:51:08 PDT 2020
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
More information about the llvm-commits
mailing list