[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