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

Florian Hahn via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun May 1 12:17:52 PDT 2022


fhahn added a comment.

> @fhahn Remind me - why did you want me to split these? If we're initially just going for integer support, can't both be done at the same time in this patch?

I think my original thinking was that `__builtin_reduce_mul` isn't defined at the moment https://clang.llvm.org/docs/LanguageExtensions.html#vector-builtins

So I think it would be good to either the update the docs  in the patch (or preferably be split off the straight forward `__builtin_reduce_add` change :))


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