[PATCH] D30086: Add generic IR vector reductions

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 19 10:25:57 PST 2017


mehdi_amini added a comment.

In https://reviews.llvm.org/D30086#681167, @delena wrote:

> According to the code that I see, all reduction intrinsics have the same form and the same handling.
>  IMO, llvm.vector.reduce(<metadata>, <vector>) would cover all aspects of FP modes, and integer signed-unsigned variants.
>  The <metadata> class can include opcode and "properties".


What is the advantage of this?

I mean, we don't do our scalar instructions as   `binop <metadata>, op1, op2`?


Repository:
  rL LLVM

https://reviews.llvm.org/D30086





More information about the llvm-commits mailing list