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