[PATCH] D73854: [ARM] Expand vector reduction intrinsics on soft float

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 3 00:44:43 PST 2020


nikic added a comment.

In D73854#1854081 <https://reviews.llvm.org/D73854#1854081>, @dmgreen wrote:

> Because we try to soften the vecreduce before we expand it?


That's right. It's actually possible to do an expansion during softening with a little hack (using `ReplaceValueWith` and returning `SDValue()` you can get recursive return value type legalization -- not sure if that's an intended feature, but it seems to work...) I do plan to submit some followup patches to add the missing legalization bits I ran into.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73854





More information about the llvm-commits mailing list