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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
