[PATCH] D157757: [Headers] Replace __need_STDDEF_H_misc with specific __need_ macros

Ian Anderson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 11 15:12:17 PDT 2023


iana marked an inline comment as done.
iana added inline comments.


================
Comment at: clang/lib/Headers/stddef.h:112
 #if defined(_MSC_EXTENSIONS) && defined(_NATIVE_NULLPTR_SUPPORTED)
-namespace std { typedef decltype(nullptr) nullptr_t; }
-using ::std::nullptr_t;
+// __need_NULL would previously define nullptr_t for C++, add it here.
+#define __need_nullptr_t
----------------
efriedma wrote:
> I think this code was pulled into the `__need_NULL` conditional by accident; I doubt anyone actually depends on the precise interaction here.  Just move to the same place as the other `__need_` defines.
👍 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D157757/new/

https://reviews.llvm.org/D157757



More information about the cfe-commits mailing list