This revision was automatically updated to reflect the committed changes.
Closed by commit rG8a92c45e07dc: [Clang] Add integer mul reduction builtin
(authored by RKSimon).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117829/new/
https://reviews.llv
RKSimon added a comment.
ping @scanon
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
fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.
LGTM, thanks!
IIRC @scanon had some opinions about `__builtin_reduce_mul` during some earlier
discussions. Please wait a few days in case there are additional comments.
Repository:
rG LLVM G
RKSimon updated this revision to Diff 426367.
RKSimon added a comment.
Add "__builtin_reduce_mul" to the docs listing supported reductions
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117829/new/
https://reviews.llvm.org/D117829
Files:
clang/do
RKSimon updated this revision to Diff 426366.
RKSimon retitled this revision from "[Clang] Add integer add/mul reduction
builtins" to "[Clang] Add integer mul reduction builtin".
RKSimon edited the summary of this revision.
RKSimon added a comment.
Just handle the mul case now that D124741