[PATCH] D117480: [IR] Extend llvm.vector.reduce.fadd
Jun Zhang via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 17 06:56:01 PST 2022
junaire 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.
Thanks to have a look at this!
The relevant context is here: https://reviews.llvm.org/D116736
> 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?
Well, maybe I didn't get the reviewer's ideas right? Any suggestions are appreciated!
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