[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