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

Asiri Rathnayake via cfe-commits cfe-commits at lists.llvm.org
Sun Sep 25 14:04:36 PDT 2016


rmaprath added a subscriber: rmaprath.
rmaprath added a comment.

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>`?

You will also need a test for the new header, https://reviews.llvm.org/D12747 has examples.

Hope this helps!

/ Asiri


https://reviews.llvm.org/D24903





More information about the cfe-commits mailing list