EricWF added a comment. In https://reviews.llvm.org/D33082#765644, @martell wrote:
> LGTM but I can't speak for the area where you added `#include <cstdio>` and > killed off the `_NEWLIB_VERSION` check > Seems in order based on > https://sourceware.org/ml/newlib-cvs/2014-q3/msg00038.html > Maybe make a note of the minimum newlib version supported somewhere? I removed the section you're referring to because it's unneeded. `<__locale>` does the exact same `#include` dance. and `<locale>` includes `<__locale>`. It should not have an affect on functionality. https://reviews.llvm.org/D33082 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits