arichardson abandoned this revision.
arichardson added a comment.
Herald added a subscriber: jrtc27.
Herald added a project: All.
Hopefully no longer required.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D44604/new/
https://reviews.llvm.org/D44604
_
bsdjhb added a comment.
FWIW, I ended up fixing FreeBSD to only use in freestanding
environments and always use in userland which makes this patch no
longer necessary. (Only one place needed to be fixed.)
Repository:
rC Clang
https://reviews.llvm.org/D44604
___
krytarowski added a comment.
This does not fix the potential problem on NetBSD. We define
`__VA_LIST_DECLARED` (with extra `_` on the beginning).
The proper solution looks to go for:
#if __STDC_HOSTED__ && __has_include_next()
Repository:
rC Clang
https://reviews.llvm.org/D44604
_
dim added a comment.
I'm fine with either this approach, or with the approach in stdint.h, e.g.
using `__has_include_next()`, and in that case first including the
system-provided header before this one.
Repository:
rC Clang
https://reviews.llvm.org/D44604
arichardson created this revision.
arichardson added reviewers: efriedma, dim.
Herald added subscribers: cfe-commits, krytarowski, sdardis, emaste.
On FreeBSD it is currently not possible to use the clang builtin
headers to build the base system. The build will fail with the
following error if I d