Re: [PATCH] D16634: [libcxx] Whitelist inclusion of sysctl.h instead of blacklisting

2016-01-28 Thread Ben Craig via cfe-commits
bcraig added a comment. I plan on submitting this tomorrow morning (my time) so that I have lots of time to monitor build failures. http://reviews.llvm.org/D16634 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin

Re: [PATCH] D16634: [libcxx] Whitelist inclusion of sysctl.h instead of blacklisting

2016-01-28 Thread Ben Craig via cfe-commits
bcraig updated this revision to Diff 46292. bcraig added a comment. CloudABI doesn't define unix, so stop blacklisting it explicitly and let the whitelist filter it out. http://reviews.llvm.org/D16634 Files: src/thread.cpp Index: src/thread.cpp ==

Re: [PATCH] D16634: [libcxx] Whitelist inclusion of sysctl.h instead of blacklisting

2016-01-27 Thread Marshall Clow via cfe-commits
mclow.lists accepted this revision. mclow.lists added a comment. This revision is now accepted and ready to land. This looks fine to me. http://reviews.llvm.org/D16634 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi

Re: [PATCH] D16634: [libcxx] Whitelist inclusion of sysctl.h instead of blacklisting

2016-01-27 Thread Ben Craig via cfe-commits
bcraig marked 2 inline comments as done. bcraig added a comment. http://reviews.llvm.org/D16634 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D16634: [libcxx] Whitelist inclusion of sysctl.h instead of blacklisting

2016-01-27 Thread Ben Craig via cfe-commits
bcraig added a comment. Added cfe-commits to subscriber list. http://reviews.llvm.org/D16634 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits