[PATCH] D117480: [IR] Extend llvm.vector.reduce.fadd

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 17 07:00:35 PST 2022


fhahn added a comment.

In D117480#3248421 <https://reviews.llvm.org/D117480#3248421>, @nikic wrote:

> I don't get this change. We already use reassoc FMF to allow a non-ordered reduction, what's the purpose of the new flag?
>
> This also needs a LangRef update.

`reassoc` implies 'any order', but in some cases it is desirable to specify a specific order, e.g. for the vector reduction builtin provided by Clang.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117480



More information about the llvm-commits mailing list