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

Simon Pilgrim via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 17 07:02:38 PDT 2022


RKSimon added a comment.

In D117829#3386398 <https://reviews.llvm.org/D117829#3386398>, @fhahn wrote:

> @RKSimon are you planning on pushing this patch through? From my perspective, it looks good with splitting off `__builtin_reduce_mul` and adding a TODO to also implement it for floating point types.

You mean handle add/mul in separate patches? I'm happy to continue with this, I just didn't want to make it more difficult for us to add fp support in the future - particularly if we do end up needing the scalar initial_value argument


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