[PATCH] D117829: [Clang] Add integer mul reduction builtin

2022-05-09 Thread Simon Pilgrim via Phabricator via cfe-commits
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

[PATCH] D117829: [Clang] Add integer mul reduction builtin

2022-05-06 Thread Simon Pilgrim via Phabricator via cfe-commits
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

[PATCH] D117829: [Clang] Add integer mul reduction builtin

2022-05-05 Thread Florian Hahn via Phabricator via cfe-commits
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

[PATCH] D117829: [Clang] Add integer mul reduction builtin

2022-05-02 Thread Simon Pilgrim via Phabricator via cfe-commits
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

[PATCH] D117829: [Clang] Add integer mul reduction builtin

2022-05-02 Thread Simon Pilgrim via Phabricator via cfe-commits
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