[PATCH] D24903: [libcxx] [include] Declare __STDC_*_MACROS for C++11 compat in old libc

Michał Górny via cfe-commits cfe-commits at lists.llvm.org
Sun Sep 25 23:46:06 PDT 2016


mgorny added a comment.

In https://reviews.llvm.org/D24903#551890, @rmaprath wrote:

> When the `<cfoo>` headers where split into `<cfoo>` and `<foo.h>` headers in https://reviews.llvm.org/D12747, we have left out `<cstdint>` and `<ctime>`. I was going to put up a patch to fix this but couldn't get to it. It's good that you are adding `<stdint.h>`, perhaps you can follow what was done in https://reviews.llvm.org/D12747 and repeat the same for `<cstdint>`?


I'm sorry but could you be a little more specific, please? I've looked into that diff, and it seems that the cstdint resembles e.g. cinttypes already -- i.e. there's only the include, and 'using' lines there. Or do you mean that I should copy/update the synopsis over to stdint.h?


https://reviews.llvm.org/D24903





More information about the cfe-commits mailing list