EricWF accepted this revision. EricWF added a comment. This revision is now accepted and ready to land.
LGTM modulo inline comments. ================ Comment at: include/new:129 @@ +128,3 @@ +/* + * NOTE: When using the Microsoft CRT, the operator new and delete + * family of functions are defined statically in msvcrt.lib. Marking ---------------- Could you put this documentation in `VisibilityMacros.rst` instead of in the headers? https://reviews.llvm.org/D25042 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits