[PATCH] D157793: [Headers] Add missing __need_ macros to stdarg.h
James Y Knight via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Aug 15 09:45:52 PDT 2023
jyknight added a comment.
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)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157793/new/
https://reviews.llvm.org/D157793
More information about the cfe-commits
mailing list