[PATCH] D152370: [Intrinsic] Introduce reduction intrinsics for minimum/maximum

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

In D152370#4403718 <https://reviews.llvm.org/D152370#4403718>, @anna wrote:

> I'm just curious what does relying on legalization helps with? Is it just a cleaner approach?

Basically yes. It makes it easy to custom lower for targets that have native reductions and produces a sensible tree legalization (that is easy to isel match) otherwise. The shuffle representation is the only way we have in IR, but it's not a convenient representation in DAG.


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

https://reviews.llvm.org/D152370