[PATCH] D140250: Define NULL in its own header

Richard Smith - zygoloid via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 25 12:26:41 PST 2023


rsmith added a comment.

Wait a second... if an OS wants only `NULL`, we already have a supported way of achieving that, which is compatible with GCC and glibc and other POSIX compilers -- define `__need_NULL` before including the header.

We shouldn't be providing internal headers with `__mangled_names` for OSes to include, those should be implementation details.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140250



More information about the cfe-commits mailing list