[PATCH] D45336: Apply accumulator to fadd/fmul experimental vector reductions (PR36734)

[Steve Canon via Phabricator via llvm-commits]

scanon added a comment.

> I also wonder whether requiring fast-math to allow tree reductions is overkill. Tree reductions can be implemented reasonably efficiently in many architectures, while linearly ordered reduction appear to me to be more of a niche.

Agreed. Tree-reduction is significantly faster and has smaller average error; it should really be the default in both LLVM and source languages except where there's an explicit constraint that forces linear reduction.


Repository:
  rL LLVM

https://reviews.llvm.org/D45336