[PATCH] D117829: [Clang] Add integer add/mul reduction builtins

Craig Topper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 20 16:23:43 PST 2022


craig.topper added a comment.

In D117829#3259588 <https://reviews.llvm.org/D117829#3259588>, @RKSimon wrote:

> I should mention - according to https://clang.llvm.org/docs/LanguageExtensions.html `__builtin_reduce_add()` already exists, which I don't think is true.

Does that documentation need to be adjusted to say integer only? Is our intention to use __builtin_reduce_add for both integer and fp in the future or would we have __builtin_reduce_fadd?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117829



More information about the cfe-commits mailing list