mclow.lists accepted this revision. mclow.lists added a comment. This revision is now accepted and ready to land.
I can't vouch for the correctness (not having a sun system to play with), but this LGTM. at some point, the repeated expression `!(defined(_LIBCPP_MSVCRT) || defined(_AIX) || defined(__sun__))` will get unwieldy enough that we'll want to give it a name, but that's for another patch. Do you need someone to commit this for you? https://reviews.llvm.org/D23507 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits