iana added a comment. In D157793#4588909 <https://reviews.llvm.org/D157793#4588909>, @jyknight wrote:
> I'm a bit confused by this change vs its description. > > It looks like stdarg already supported `__need___va_list`, which is what you > said Apple needs. Does Apple //also// require the other __need_va_copy/etc, > too? If not, why add support for them? (I think we should prefer not to add > support for pieces that are not actually needed) That name is misleading, `__need___va_list` gives you `__gnuc_va_list` and not `va_list`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157793/new/ https://reviews.llvm.org/D157793 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
