[PATCH] D21553: Do not require __STDC_LIMIT_MACROS and others

[James Y Knight via llvm-commits]

jyknight added a comment.

In http://reviews.llvm.org/D21553#464402, @chfast wrote:

> In http://reviews.llvm.org/D21553#463304, @jyknight wrote:
>
> > Not requiring the defines is good; removing the defines in the build files is not. We still want to be able to build out of the box on systems with this known brokenness.
>
>
> Can we identify such broken systems in the first place?


Looking around briefly at the libcs I have handy...
glibc stopped checking them in 2013, newlib dropped them in 2015, musl in 2013, bionic in 2014. (dunno release dates, those are commit dates). So things look pretty good there.

On the other hand, you have platforms like freebsd and netbsd, which apparently *still* haven't removed these #ifdefs from their stdint.h. So...yeah, seems like there's still a need for the defines, for now. Anyone over in BSD land paying attention?... /me pokes joerg. :)


http://reviews.llvm.org/D21553