Redefine NULL to nullptr
authorIlya Leoshkevich <iii@linux.ibm.com>
Mon, 29 Jun 2020 18:36:03 +0000 (20:36 +0200)
committerIlya Leoshkevich <iii@linux.ibm.com>
Thu, 2 Jul 2020 10:38:18 +0000 (12:38 +0200)
commitd59a576b8b5e12c3a56f0262912090e2921f5daa
treed615a216d6a34f5d1ebc2deeef15e8d3e2714b70
parentd0e7c73c515c305863620a821ee85bc557bfbef5
Redefine NULL to nullptr

Bootstrap with musl libc fails with numerous "missing sentinel in
function call" errors.  This is because musl defines NULL as 0L for C++,
but gcc requires sentinel value to be a pointer or __null.

Jonathan Wakely says:

    To be really safe during stage 1, GCC should not use NULL as a
    pointer sentinel in C++ code anyway.

    The bootstrap compiler could define it to 0 or 0u, neither of which
    is guaranteed to be OK to pass as a varargs sentinel where a null
    pointer is expected.  Any of (void*)0 or (void*)NULL or nullptr
    would be safe.

While it is possible to fix this by replacing NULL sentinels with
nullptrs, such approach would generate backporting conflicts, therefore
simply redefine NULL to nullptr at the end of system.h, where it would
not confuse system headers.

gcc/ChangeLog:

2020-06-30  Ilya Leoshkevich  <iii@linux.ibm.com>

PR bootstrap/95700
* system.h (NULL): Redefine to nullptr.
gcc/system.h