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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to