[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
Mon Sep 26 00:07:16 PDT 2016


mgorny marked an inline comment as done.
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>`?
>
> You will also need a test for the new header, https://reviews.llvm.org/D12747 has examples.


I see that libcxx already has a pretty complete test for stdint.h.

After consulting the standard, I've shamelessly copied the C part of synopsis to my stdint.h.


https://reviews.llvm.org/D24903





More information about the cfe-commits mailing list