[PATCH] D25361: [libcxx] Add the missing limits.h header

2016-10-10 Thread Asiri Rathnayake via cfe-commits
rmaprath closed this revision. rmaprath added a comment. Thanks. Committed as r283726. https://reviews.llvm.org/D25361 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D25361: [libcxx] Add the missing limits.h header

2016-10-07 Thread Richard Smith via cfe-commits
rsmith added a comment. I only skipped this one because it seemed pointless to me. I have no objection to providing it, and this change makes perfect sense if libc++ wants to guarantee that `<__config>` is included whenever any standard C++ header is included. https://reviews.llvm.org/D25361

[PATCH] D25361: [libcxx] Add the missing limits.h header

2016-10-07 Thread Marshall Clow via cfe-commits
mclow.lists added a comment. This looks fine to me, but since @rsmith added all the other C headers, I'd like him to weigh in. https://reviews.llvm.org/D25361 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mai

[PATCH] D25361: [libcxx] Add the missing limits.h header

2016-10-07 Thread Asiri Rathnayake via cfe-commits
rmaprath created this revision. rmaprath added reviewers: EricWF, mclow.lists, rsmith. rmaprath added a subscriber: cfe-commits. The implementation of [depr.c.headers] in https://reviews.llvm.org/D12747 introduced the necessary C headers into libc++. This patch adds one more missing headers: limi