[PATCH] D25042: [libc++] Clarify _LIBCPP_NEW_DELETE_VIS for Windows

Eric Fiselier via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 28 14:28:32 PDT 2016


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





More information about the cfe-commits mailing list