[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